feat: Implemented Mark for Review system for student self-regulation#964
Closed
Boltu-007 wants to merge 2329 commits into
Closed
feat: Implemented Mark for Review system for student self-regulation#964Boltu-007 wants to merge 2329 commits into
Boltu-007 wants to merge 2329 commits into
Conversation
…/vibe into feature/hp-system
…usactivelearning/vibe into feature/hp-system
… HP caps optional, PDF only, backend errors, past dates blocked
…uousactivelearning/vibe into feature/hp-system
…ousactivelearning/vibe into feature/hp-system
…ntinuousactivelearning/vibe into feature/hp-system
…uousactivelearning/vibe into feature/hp-system
…tivelearning/vibe into feature/hp-system
…usactivelearning/vibe into feature/hp-system
…icharanashala#903) * Added fixed for the linear progression disabled course and Gurusetu. * latest changes incorporated.. --------- Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
* Fix: When a student is ejected and if the policy was updated while the student was ejected, then the student won't get the "re-acknowledge" policy modal when they're re-instated. * Fix: Course was being shown even when the course policy required re-acknowledgement * Fix: Re-acknowledge button in the notifications kept on showing the button even after re-acknowledgement is done. * Fix: When a cohort policy was changed and updated twice or more, all the notifications showed the re-acknowledgement button but only the latest one should have shown it. --------- Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
* feat:added course and cohort details on ejection appeal * feat:added confirmation for approve and reinstate appeal --------- Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
…nashala#883) * fix: Prevented duplicate watchtime document creation in quiz * fix: handled watchtime issue in quiz --------- Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
* fix:showing all events in history tab * fix:showing events in same start and end date --------- Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
* - Refactor: Service and repo layer (autoejectionengine) - Corrected cron job run time - Commented out inactivity trigger from AutoEjectionEngine * Refactor: Service and repo layer(ejectionpolicyservice) * Refactor: removed inactivity trigger from UI Refactor: removed service code for inactivity trigger * - chore: removed logs from files i have worked on - refactor: Moving Appeal badge logic to a different new component for performance(re-rendering) - refetch: invalidating appeals on submit of appeals, approval or rejection of appeals --------- Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
vicharanashala#873) Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
Add course + version details in cohort card Co-authored-by: Abiram-k <abiramk0107@gmail.com> Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
…la#905) * chore: add a modal when all quiz attempts are exhausted * fix: clicking on already completed quiz keeps loading * minor: resolve conflicts * chore: removed extra logs --------- Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com>
* Added fixed for the linear progression disabled course and Gurusetu. * latest changes incorporated.. * Worked on the teachers side. Now showing completed item and total items. * feat: change modal text and action name * fix(backend): change minimum required percent to 15% * Worked on the start and stop now student tries to skip video it is forced to see video again. * Now if student tries to skip video. So rather then skipping he has to rewatch the video... --------- Co-authored-by: Nandan Prabhudesai <nandanprabhudesai@gmail.com> Co-authored-by: Abiram-k <abiramk0107@gmail.com>
…#914) Fixed Quiz Evaluation Mongo Instance Comparing logic to maintain consistency
github-actions
Bot
added
docs
Author
|
Sorry, i created this PR by mistake. Closing this in favour of PR #965, which combines both the Mark for Review and Return to Video features into one clean merge. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note to Reviewer: This is a stacked PR that builds on top of PR #934. It currently includes the commits from #934. Once #934 is merged, this diff will automatically resolve to show only the new Mark for Review logic.
Description :-
This PR introduces a personal 'Mark for Review' feature, distinct from the administrative 'Flag' system.
Key Changes:
Created a persistent global state using Zustand and localStorage to track flagged items.
Added a toggleable bookmark UI in both the Quiz and Video components.
Implemented a 'Marked for Review' dashboard tab for non-linear revision.
Applied stability fixes to ensure reliable rendering.