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

Add functionality to manage favorite doctors (add/remove) #193

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

23nosurrend
Copy link
Collaborator

What does this pull request do?

Adds functionality to allow users to add and remove doctors from their favorites list.

Description

Implemented the following functionalities:

  • Add Doctor to Favorites: Users can now add doctors to their favorites list by clicking on a heart icon associated with each doctor.

  • Remove Doctor from Favorites: Users can remove doctors from their favorites list by clicking on the heart icon again, which triggers a confirmation popup.

  • Supabase Integration: The favorites data is managed through Supabase, ensuring seamless addition and removal of doctors from the user's favorites list.

  • UI Updates: The UI reflects the current state of the doctor's favorite status, updating in real-time when a doctor is added or removed from the favorites list.

Screenshots

Screenshot_20240705_104930_Expo Go
Screenshot_20240705_104914_Expo Go

@23nosurrend 23nosurrend added the Need Review Requires reviews label Jul 5, 2024
@23nosurrend 23nosurrend added this to the Integrate Key App Features milestone Jul 5, 2024
@23nosurrend 23nosurrend self-assigned this Jul 5, 2024
@23nosurrend 23nosurrend linked an issue Jul 5, 2024 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Jul 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.22%. Comparing base (c86e3df) to head (3f41b08).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #193   +/-   ##
========================================
  Coverage    72.22%   72.22%           
========================================
  Files            3        3           
  Lines           18       18           
  Branches         3        3           
========================================
  Hits            13       13           
  Misses           5        5           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@23nosurrend 23nosurrend force-pushed the ft-add-remove-favoriteDoctors branch from 23b1e67 to 9372c1a Compare July 7, 2024 21:27
@23nosurrend 23nosurrend force-pushed the ft-add-remove-favoriteDoctors branch from 9372c1a to 3f41b08 Compare July 8, 2024 09:07
@mr3nz1 mr3nz1 self-requested a review July 8, 2024 09:28
Copy link
Contributor

@Irirwanirira Irirwanirira left a comment

Choose a reason for hiding this comment

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

Good Job

@Nkbtemmy Nkbtemmy merged commit 57ba818 into develop Jul 8, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Need Review Requires reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature:Add/remove doctor from favorites
5 participants