Skip to content
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

Feature: Notifications Implementation #152

Merged
merged 1 commit into from
Jul 23, 2024
Merged

Conversation

Latiah
Copy link
Contributor

@Latiah Latiah commented Jul 21, 2024

What does this PR do?

This PR includes notification implementation where a user will be able to view notifications when an event is done like creating an appointment or cancelling an appointment

Description of Task to be completed?

Fetching data from the database

How should this be manually tested?

Login the app and on the home screen click on the notification icon and from there you will find the notifications

Any background context you want to provide?

What are the relevant pivotal tracker/Trello stories?
N/A

Screenshots (if appropriate)

Questions:
N/A

@Latiah Latiah requested a review from Emmyfrank July 21, 2024 22:43
@Latiah Latiah changed the title Feature: Notifications implementation Feature: Notifications Implementation Jul 21, 2024
@Latiah Latiah self-assigned this Jul 21, 2024
@Latiah Latiah added the Feature This is a project feature label Jul 21, 2024
@Latiah Latiah added this to the Integrate Key App Features milestone Jul 21, 2024
@Latiah Latiah linked an issue Jul 21, 2024 that may be closed by this pull request
Copy link
Contributor

@Emmyfrank Emmyfrank left a comment

Choose a reason for hiding this comment

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

Work on following comment

I think it might be better if if you work on deleting all notifications and singe notification because sometimes a patient will need to delete them

you can also update your PR

WhatsApp Image 2024-07-22 at 12 29 34

Copy link
Contributor

@Emmyfrank Emmyfrank left a comment

Choose a reason for hiding this comment

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

Now i can clear all notfications or single notfication , thanks for working on my comments

@Latiah Latiah force-pushed the ft-notifications-implementation branch 2 times, most recently from ffe2bd3 to 82ccdd8 Compare July 22, 2024 15:26
@Latiah Latiah force-pushed the ft-notifications-implementation branch 2 times, most recently from 76ae066 to 4308d36 Compare July 22, 2024 21:00
@Latiah Latiah force-pushed the ft-notifications-implementation branch from 4308d36 to c417fb0 Compare July 22, 2024 21:02
Copy link
Contributor

@justinmihigo justinmihigo left a comment

Choose a reason for hiding this comment

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

Notifications are coming but if you can manage so that you can use system notifications so that a user can know immediately. but otherwise great job on this

@Latiah
Copy link
Contributor Author

Latiah commented Jul 23, 2024

Notifications are coming but if you can manage so that you can use system notifications so that a user can know immediately. but otherwise great job on this

thanks, @justinmihigo I am on it

@Adadie Adadie merged commit efa44a3 into dev Jul 23, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This is a project feature Need Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Notifications Implementation
5 participants