Skip to content

XXXX taking over for Manvitha Phase 4 – Timer Functionality#1916

Open
myeeli wants to merge 6 commits into
developmentfrom
Manvitha-Student-activityLog-Timer
Open

XXXX taking over for Manvitha Phase 4 – Timer Functionality#1916
myeeli wants to merge 6 commits into
developmentfrom
Manvitha-Student-activityLog-Timer

Conversation

@myeeli

@myeeli myeeli commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

Description

image image

Related PRS (if any):

To test this backend PR you need to checkout the #4432 frontend PR.

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. go to localhost:5173/studentportal

Screenshots or videos of changes:

1.mp4

Note:

Include the information the reviewers need to know.

@myeeli myeeli changed the title Manvitha student activity log timer Manvitha Phase 4 – Timer Functionality Dec 6, 2025

@Anusha-Gali Anusha-Gali 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 Manvitha,

I have reviewed your PR locally, though the main functionality does work as per requirement. The page is not in support of Dark Mode. Do let me know if that is not a part of the requirement.
Screenshot 2025-12-19 at 6 42 01 PM
Screenshot 2025-12-19 at 6 41 55 PM
Screenshot 2025-12-19 at 6 42 42 PM

@VijayAnirudh VijayAnirudh 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.

@rohanrastogi311 rohanrastogi311 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 Manvitha, I have reviewed your PR have posted my comments on the frontend pr (4432): OneCommunityGlobal/HighestGoodNetworkApp#4432 (review)

@SharadhaKasiviswanathan

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request

Resolved the merge conflicts by merging origin/development into this branch and fixing the conflict in src/startup/routes.js.

Commit: a35e17c

@sonarqubecloud

Copy link
Copy Markdown

@kzou55 kzou55 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.

I ran and tested the PR locally alongside its associated frontend,

I left the review in the frontend PR

@one-community one-community changed the title Manvitha Phase 4 – Timer Functionality XXXX taking over for Manvitha Phase 4 – Timer Functionality Jul 18, 2026
@one-community one-community added the Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. label Jul 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants