Skip to content

feat: Add emergency button to caregiver dashboard and STML user screen, integrate notification service#228

Closed
Layesuza wants to merge 14 commits intodeveloperfrom
feature/emergency-button
Closed

feat: Add emergency button to caregiver dashboard and STML user screen, integrate notification service#228
Layesuza wants to merge 14 commits intodeveloperfrom
feature/emergency-button

Conversation

@Layesuza
Copy link

Description

This PR adds an emergency button to the caregiver dashboard and the STML user screen. It also integrates a notification service to handle alerts.

Changes Made

  • Added the emergency button to the following screens:
    • caregiver_dashboard
    • stml_user_screen
  • Integrated the notification service (notification_service.dart)
  • Created a help service (help_service.dart)
  • Added VS Code configuration files (.vscode/launch.json, .vscode/settings.json)

Additional Information

  • VS Code configuration files were added to facilitate debugging.
  • No additional external dependencies were added.

@Layesuza Layesuza added the STML Sp25 Label for Spring 2025 STML group (Team C and D) label Mar 20, 2025
@ashekh9
Copy link
Contributor

ashekh9 commented Mar 26, 2025

I was able to run the app but when clicking on the button, nothing works, I get an error stating that it could not launch the number you provided. I think the canLaunch() method that you're using in the caregiver-dashboard.dart file is also not the right one since that function has been deprecated now so it may not work with that function. Another thing is that, for some reason, your branch is running Team A and B's Github actions check and this is the first time I've seen it for our PRs. Just check and see to make sure you're not messing with their code versions either with this merge.

Copy link
Contributor

@aholt014 aholt014 Mar 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

drop this commit - doesn't need to be merged

@aholt014 aholt014 self-requested a review March 29, 2025 02:52
@aholt014
Copy link
Contributor

DO NOT MERGE. Doing so will delete Team A and B work :/

@aholt014
Copy link
Contributor

I was able to run the app but when clicking on the button, nothing works, I get an error stating that it could not launch the number you provided. I think the canLaunch() method that you're using in the caregiver-dashboard.dart file is also not the right one since that function has been deprecated now so it may not work with that function. Another thing is that, for some reason, your branch is running Team A and B's Github actions check and this is the first time I've seen it for our PRs. Just check and see to make sure you're not messing with their code versions either with this merge.

since the functionality isnt there i am okay w/ rejecting this PR and/or leaving it out for this release due to time esp if it isnt a must have requirement.

@aholt014 aholt014 closed this Mar 29, 2025
@aholt014
Copy link
Contributor

NOTE: as to why it is being rejected
"has Team A/B contents and other files that do not need to be pushed/merged; need to just create a new PR and add changes Team C/D manually, the functionality is not working properly so this will not make it in next release, also the functionality is already in the code - rejected and closed as the functionality is already in the code"

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

Labels

STML Sp25 Label for Spring 2025 STML group (Team C and D)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants