-
-
Notifications
You must be signed in to change notification settings - Fork 122
feat: Add Emergency Help button and local resource finder (issue #5) #27
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
feat: Add Emergency Help button and local resource finder (issue #5) #27
Conversation
…mproved global resources display
…onality, removed unused imports
@eccentriccoder01 @pkbros found more merge conflicts as of now, mainly with the chat persistence feature. I have resolved it, but you all will not find it in the demo link because the code is deployed from my fork. |
@Baibhab-Adhikari Have you resolved the conflicts arising from the latest merged PR #20? Rest everything seems cool... |
@eccentriccoder01 yessir, that was the last conflict I resolved it. |
@Baibhab-Adhikari Can you share a latest deployment link to it? |
@eccentriccoder01 sure, I am at college right now, I will deploy the latest changes and send you the link by tonight. |
Sure @Baibhab-Adhikari, take your time! Also try to keep the emergency button RED as it was before, it suits the button more in my opinion... |
Sure, I agree with the button colour, I might have accidentally changed that. I'll fix it and will provide you the deployment link. And yes, I forgot about linking the issue. I'm on it |
@eccentriccoder01 button colour is fixed and deployment is ready to go! Here it is: Fixed Demo URL. Please do let me know if there are any other things to check. If this is merged, I'll start working on issue #11 |
Thank you so much for this valuable and well-executed contribution @Baibhab-Adhikari!
Now you can work on the other issues, thanks again for your patience and efforts in improving TalkHeal❕ |
Thank you so much @eccentriccoder01 for letting me contribute to your project (My first contribution ever lol). Thank you! Also since I am quite busy at college right now, due to some events, can you give me some more time to work on issue #11 ? Deadline at Tuesday (29th July) would be amazing.. Please do let me know, thank you! |
@Baibhab-Adhikari there is no hurry, I know you're one of the sincere contributors... No issues even if you need time post deadline, take your time... Also, it is genuinely impressive if it's your first contribution, since you're already working like a Pro, LOL...All the best❕ |
really appreciate it @eccentriccoder01 🙏 |
This pull request aims to implement the "Emergency Help Button" functionality to the TalkHeal application (Closes #5 ), refactors existing emergency resources, and improves code readability. The most significant changes include adding a dedicated emergency page (emergency_page.py in components directory) with map embed (OpenStreetMap), restructuring global crisis resources, and removing redundant functionality from the header (in header.py).
Emergency Help Feature:
TalkHeal.py
: Added session state variables forshow_emergency_page
and implemented page routing logic to display the emergency help page when activated.components/emergency_page.py
: Created a new emergency help page with functionality for location-based search, map display, and dynamic Google search links for mental health crisis support.Refactoring Emergency Resources:
components/sidebar.py
: Replaced hardcoded emergency contacts with a structuredGLOBAL_RESOURCES
list to improve maintainability and added a button to navigate to the emergency page. [1] [2]components/sidebar.py
: Updated the "Crisis Support" section to useGLOBAL_RESOURCES
for displaying global crisis hotlines. (Merge Conflict resolved here)Code Cleanup and Redundancy Removal:
components/header.py
: Removed redundant "Find Help Nearby" functionality from the header, as it overlaps with the new emergency page.core/config.py
: Fixed an error in API key configuration for the Gemini model.Demo Link: TalkHeal - Emergency Button
Please do let me know if any changes are required.Thanks! @eccentriccoder01 @pkbros