Skip to content

Shravya: Phase 4 - Tagged Logs for Young Learner Support - Frontend#4381

Open
ShravyaKudlu wants to merge 9 commits into
developmentfrom
Phase4/Tagged_Logs_for_Young_Learner_Support/Frontend
Open

Shravya: Phase 4 - Tagged Logs for Young Learner Support - Frontend#4381
ShravyaKudlu wants to merge 9 commits into
developmentfrom
Phase4/Tagged_Logs_for_Young_Learner_Support/Frontend

Conversation

@ShravyaKudlu

@ShravyaKudlu ShravyaKudlu commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

Description:

Implements Phase 4 - Tagged Logs for Young Learner Support - Frontend
The UI will clearly identify log entries that were created or edited by a Learning Support Team member.
Features:
When an educator views a student's 'DailyLogPage', any log entry assisted by a support member will display a clear visual tag (e.g., "Assisted by: [Support Member Name]").
When a Young Learner Support member creates a log entry, the UI may or may not show a specific indicator, as they are acting on behalf of the student. The tagging is primarily for the educator's view.

Frontend Tasks:
URL: (Modify existing) '/educator/daily-log/:studentId'.
Update the 'LogItemCard.jsx' component to conditionally render an "Assisted" tag or byline based on the new API response field.
Add a new CSS module style for the "assisted" tag.
image

Figma Design provided:
swappy-20251115-144455

Related PRS (if any):

This frontend PR is related to the development backend.

Main changes explained:

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to path ActivityLogs/DailyLog
  6. verify All the buttons and functionality works as expected
  7. verify this new feature works in [dark mode)
  8. Reference the video below

Screenshots or videos of changes:

Test.mp4

Note:

Thia is is of Mock Data

@netlify

netlify Bot commented Nov 13, 2025

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit ca1b52d
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a378480b4fae30008826433
😎 Deploy Preview https://deploy-preview-4381--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud

Copy link
Copy Markdown

@ShravyaKudlu ShravyaKudlu marked this pull request as ready for review November 15, 2025 19:55
@ShravyaKudlu ShravyaKudlu changed the title Shravya/Feature/Phase4/Tagged_logs_for_young_learner_support/frontend Shravya: Phase 4 - Tagged Logs for Young Learner Support - Frontend Nov 25, 2025
@beblicarl beblicarl self-requested a review December 19, 2025 19:33
beblicarl
beblicarl previously approved these changes Dec 19, 2025

@beblicarl beblicarl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This functionality works as intended and the code is great

https://www.loom.com/share/9752cb754b2c4762ab4ab55aed42f0b2

@sonarqubecloud

sonarqubecloud Bot commented May 1, 2026

Copy link
Copy Markdown

@ShravyaKudlu ShravyaKudlu added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label May 31, 2026

@DeepighaJ DeepighaJ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked out to current branch and tried to start frontend npm run start:local and getting error code: 'MODULE_NOT_FOUND'

Image

@HemanthNidamanuru HemanthNidamanuru left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
I tested this locally following the steps provided and found 2 issues:

Student name is showing "Young Learner" placeholder instead of the actual student name.
Tags need styling improvement in dark mode.

Image Image Image

@DeepighaJ DeepighaJ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Checked out to current branch and tested the PR locally.
  • In Dark mode the Assisted by Name is not visible properly.
  • Print View button is not clickable.
  • Student is able to create log and submit to teacher and teacher can view it and send feedback as expected.
  • Support user can log support as expected.
  • Functionality works as expected.
Image Image Image Image Image Image Image Image

@ShravyaKudlu

Copy link
Copy Markdown
Contributor Author

Thanks for your feedback.

@HemanthNidamanuru ,

  1. could you please confirm again, I'm not sure what the issue is, I have pasted the requirements mentioned for more clarity, since sara is a supporter of young learners, I assumed, she would be assisting someone placeholder, so if we could integrate it to backend, maybe they would have a set have young learners they could assist, but since its not integrated, I made it like this
  2. I have addressed the dark mode issue, could you please verify?
    @DeepighaJ
  3. print view is not mentioned in the UI, so I removed it
  4. I have addressed the dark mode issue, could you please verify

Thanks

image

@ShravyaKudlu ShravyaKudlu force-pushed the Phase4/Tagged_Logs_for_Young_Learner_Support/Frontend branch from bd540ee to 7557c1f Compare June 14, 2026 07:21

@DeepighaJ DeepighaJ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested the PR, the dark mode issue still exist, please refer the last screenshot.

Image Image Image

@sonarqubecloud

Copy link
Copy Markdown

@ShravyaKudlu

Copy link
Copy Markdown
Contributor Author

@DeepighaJ , I tested in 2 browsers, now, I fixed could you please check again

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants