daily-code icon indicating copy to clipboard operation
daily-code copied to clipboard

Published 20 hours ago • verified publisher icon code100x

bookmarks feature added and separate page to manage all tracks with s…

Open nikhilsahni7 opened this issue 1 year ago • 3 comments

…pecific pages

PR Fixes:

  • 1 added feature for users to bookmark/saved later for tracks with specific problemId/current page .

  • 2 They can watch their bookmarks on /bookmarks route which is added in profile dropdown.

  • [ ] I have performed a self-review of my code

  • [ ] I assure there is no similar/duplicate pull request regarding same issue

https://github.com/user-attachments/assets/e0d43b7f-9f13-4d99-b811-0fbbe81a13ab

image Screenshot from 2024-09-22 03-59-09

Screenshot from 2024-09-22 03-59-04

nikhilsahni7 avatar Sep 21 '24 22:09 nikhilsahni7

@devsargam please review if you have time!

nikhilsahni7 avatar Oct 05 '24 20:10 nikhilsahni7

Hey you have 8k loc changed might letting that down?

devsargam avatar Oct 05 '24 20:10 devsargam

Actually, it's a bookmark feature for a particular track, so I need to make changes in the database and create API routes to store everything. That's why the size is around 8k. I have mostly added changes in new files, so I don't think there should be any issues with merging. Also, please let me know in advance if this feature is needed.@devsargam

nikhilsahni7 avatar Oct 06 '24 08:10 nikhilsahni7