-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Feat/bookmarks #245
Feat/bookmarks #245
Conversation
moved bookmark components in their folder added isPineed missed migration by hsingh rearranged bookmark data fetching and view made edit bookmark separate and some finishing touches changed bookmark from timestamp to video
reverted seed.ts css changes
@hkirat is this the right solution? If yes, I'll start implementing other features marked as Todo |
Looks good |
I was thinking I'll implement all the other todos in the same PR, and make the UI better, or do you want to merge this first and then incrementally make it better? |
resolved the conflicts @hkirat |
/bounty $75 |
Closes #4 - Feature - Bookmarks.
Screencast.from.2024-03-19.06-07-07.webm
Todo:
@hkirat how does this look to you? There can be a button in the video player or below it as well if needed.