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

Solved Issue #238 : Favorite Button #258

Merged
merged 1 commit into from
Nov 10, 2024

Conversation

Somyajain2004
Copy link
Contributor

@Somyajain2004 Somyajain2004 commented Nov 10, 2024

The toggle feature is added to favorite button.
On toggling the card is either added or removed from the Favorites page.

Output :
Screenshot 2024-11-10 070930
Screenshot 2024-11-10 070901
Screenshot 2024-11-10 065122

As shown is the above Screenshot, Phone 2 is liked, hence it's in Favorite page, and since Phone 1 is not liked, it's not showing in Favorites page.

NOTE : fetch and use realtime user data instead of dummy data to keep the changes saved! (As mentioned in HomeScreenViewModel.kt) As, I have tried the code on temporary demo data, since I wasn't able to access realtime data.
Also, i wasn't able to signIn via gmail or Google, is there a issue with that?

The toggle feature is added to favorite button.
On toggling the card is either added or removed from the Favorites page.
NOTE : fetch and use realtime user data instead of dummy data to keep the changes saved! (As mentioned in HomeScreenViewModel.kt)
As, I have tried the code on temporary demo data, since I wasn't able to access realtime data.
@Somyajain2004 Somyajain2004 changed the title Issue #238 : Favorite Button Solved Issue #238 : Favorite Button Nov 10, 2024
@yuvrajsinghgmx yuvrajsinghgmx merged commit 386ab0c into yuvrajsinghgmx:master Nov 10, 2024
Somyajain2004 added a commit to Somyajain2004/Open-source-Practice that referenced this pull request Nov 10, 2024
PR link : yuvrajsinghgmx/ShopSmart#258

Hi GGSOC team,
The PR given mentioned above has been reviewed and merged by the reviewer. 
But, my GGSOC leaderboard score hasn't been updated for this PR. The the was a gssoc-ext, level3 problem.
Please see to it.

@sanjay-kv @MastanSayyad
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants