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/add mark as done button #1576 #1583

Conversation

SahilLamba0008
Copy link

@SahilLamba0008 SahilLamba0008 commented Nov 24, 2024

PR Fixes:

  • Added a button to manully mark content as completed.

Resolves #1576

Checklist before requesting a review

  • [☑️] I have performed a self-review of my code
  • [☑️]I assure there is no similar/duplicate pull request regarding same issue

Video Preview : https://www.loom.com/share/d94cd773a602457690712dfacd066f69?sid=19310246-6f10-4757-9c8e-f4628954caa5

@SahilLamba0008
Copy link
Author

@devsargam Could you please review and confirm if this is worth pursuing further? I will improve the code quality if this feature is needed.

@devsargam devsargam self-assigned this Nov 26, 2024
@devsargam devsargam self-requested a review November 26, 2024 15:48
@SahilLamba0008
Copy link
Author

@devsargam Just checking in, will the review take longer, or is this feature no longer needed? I’d appreciate your feedback.

@devsargam
Copy link
Collaborator

@SahilLamba0008 you are great sir. sorry I was just busy with other stuff didn't get to review this one. But it's a great feature but the db is already too complicated for such a simple application. I am closing this but this is a great pr.

Reason: Just don't want to overcomplicate the db with more migrations unless much needed :)

@devsargam devsargam closed this Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature: Add "Mark as Done" Button for Videos and Notes to Manually Track Completion
2 participants