Skip to content

Conversation

@MK-codes365
Copy link
Contributor

@MK-codes365 MK-codes365 commented Jan 24, 2026

issue: #368

Done

Production API Integration: Configured
app.config.ts
to use https://snapmap.onrender.com via Expo extra values.
Centralized Configuration: Created
src/apiConfig.ts
to centrally manage the backend URL and ensure consistency across the app.
Global Refactor: Modified
HomeScreen
, MapScreen, ProfileScreen, MyUploadsScreen, and others to dynamically read the API URL, removing all hardcoded localhost references.
Emulator Optimized: Verified setup for Android Studio Emulator testing using the Expo-managed workflow.
Integrity: Strictly followed PR requirements by keeping backend code untouched while implementing all configuration changes on the frontend.

Screenshot 2026-01-24 195615 Screenshot 2026-01-24 214021

@MK-codes365
Copy link
Contributor Author

@23abdul23 please see my pr

@MK-codes365
Copy link
Contributor Author

@23abdul23 I have submitted everything on time

@OpenGitBot
Copy link

Hey @MK-codes365

Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.

Thanks for contributing in OpenCode'25 ✨✨!

@23abdul23
Copy link
Contributor

@MK-codes365

Is the app working fully, you are able to upload photos , then see them also ??

@MK-codes365
Copy link
Contributor Author

@23abdul23 I can see photos but the camera is showing black screen when I click picture

@23abdul23
Copy link
Contributor

Obvious behaviour for Emulator

@23abdul23
Copy link
Contributor

@MK-codes365

If you can generate the .apk file also and send that to me on Discord, I'll give u extra 5 points

you have EOD for this.

@MK-codes365
Copy link
Contributor Author

@23abdul23 how much time you can give ?

@23abdul23
Copy link
Contributor

END OF DAY,

COZ ur PR is fine I can merge it now if u want

@MK-codes365
Copy link
Contributor Author

@23abdul23 no don't merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants