Sphurthy taking over for Taariq: Adding a Lesson Plan Builder UI and update routes#4978
Sphurthy taking over for Taariq: Adding a Lesson Plan Builder UI and update routes#4978TaariqMansurie wants to merge 15 commits intodevelopmentfrom
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
one-community
added
the
High Priority - Please Review First
|
I tried testing the functionalities by switching to "taariq-feature-lesson-plan-builder-frontend", but I see POST /api/educator/assign-tasks return 404 error. Not sure if I have to have any records in the database prior to testing. 
|
ManojPuttaswamy
left a comment
ManojPuttaswamy
left a comment
There was a problem hiding this comment.
please cleanup comments which aren't necessary
Anusha-Gali
left a comment
Anusha-Gali
left a comment
There was a problem hiding this comment.
Hi Taariq,
I have reviewed your PR locally and have found the below issue:
- The UI of the manually assigned and auto assigned does not show the pop of color as it does in light mode
- The calendar is in light mode when the application is in dark mode
- As you can see in the below video, there is a huge delay in terms of notification pop up which can cause people to assume that it did not get clicked and they can click it multiple times just like i did and the notification appears multiple times
- Also what exactly does 1404 mean in the notification.
- And the UI is not mobile responsive
4978.mov
one-community
changed the title
…ighestGoodNetworkApp into taariq-feature-lesson-plan-builder-frontend Made-with: Cursor # Conflicts: # package-lock.json
|
5 participants
Description
This PR adds the UI and API integration for the "Assign Lesson Plan" feature. Educators and admins can assign tasks to students manually via a button or automatically upon saving.
Implements # (PRIORITY HIGH) Phase 4 - Assign Lesson Plan - Front end and UI (WIP Taariq)
Related PRS (if any):
This frontend PR is related to the backend PR 2145
To test this frontend PR, you need to have the corresponding backend branch running locally.
…
Main changes explained:
…
How to test:
1.check into current branch
2. do yarn install and yarn start to run this PR locally
3. Clear site data/cache
4. log in as an admin or educator user
5. go to the Lesson Plan Builder page
6. verify manual assignment: Click "Assign Lesson Plan", click "Confirm" in the modal, and verify the notification appears.
7. verify auto assignment: Click "Save Lesson Plan" and verify the UI updates with the "Auto-assigned at [Time]" status.
Screenshots or videos of changes:
Note:
Include the information the reviewers need to know.