-
-
Notifications
You must be signed in to change notification settings - Fork 123
Enhance Crisis Support with GeoIP-based helpline detection #25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhance Crisis Support with GeoIP-based helpline detection #25
Conversation
Thanks for the contribution, @tilakjain619 ! The mentioned additions look really useful. But before merging, could you please:
Once this is done, we’ll proceed with the review and testing. Excellent progress! |
Merge Conflicts ResolvedUpdated the sidebar and solved the merge conflicts. |
@tilakjain619 Great, could you share the working deployment link of yours❔ |
@eccentriccoder01, just pushed it here: https://crisis-support-lgu5kyqwwvhwna64yugbkn.streamlit.app |
Hi @tilakjain619 Your PR's deployment shows just this: Could you elaborate a bit on how your pushed changes work? It's a bit unclear to me. Thanks! |
The current IP detection is picking up the server location (Streamlit’s US-based host, that's why it's showing US contacts), not the user’s actual IP. I’ll update this with browser-side geolocation or a country override. Thanks for pointing it out! |
Alright @tilakjain619, also please stay updated with the latest changes in the upstream, as the main is constantly being updated by multiple people. Thanks! |
I've resolved the merge conflicts. I updated get_user_country() to use user’s actual browser location via streamlit-js-eval. Here is deployment link: https://crisis-support-lgu5kyqwwvhwna64yugbkn.streamlit.app/ |
Great work @tilakjain619 ! I'm merging this now, thank you for all your hard work on this and efforts in improving TalkHeal... Feel free to work on more issues 💙 |
Thanks a lot! I appreciate your support. |
Uh oh!
There was an error while loading. Please reload this page.