Skip to content

Conversation

@vamsidharpanithi
Copy link

Description

This PR fixes the Time Log Visibility issue on the Member Group Check-In page by adding a Time Log Summary component and fixing multiple timer display bugs. The summary table now displays all completed time logs with member names, roles, start/stop times, and durations. Additionally, fixed issues where timers were starting from incorrect values, not preserving paused/resumed states, and clearing the time when stopped.

Fixes # (bug list priority: Medium) - Time Log Visibility Issue on Member Group Check-In Page

Related PRs (if any):

This frontend PR is related to the backend PR #1994.
To test this frontend PR you need to checkout the backend PR #1994.

Main changes explained:

How to test:

  1. Checkout the Vamsidhar-fix/time-log-visibility-frontend branch
  2. Ensure the backend PR is also checked out and running on http://localhost:4500
  3. Run npm install to ensure dependencies are up to date
  4. Start the frontend: npm start
  5. Clear browser cache/local storage
  6. Navigate to: http://localhost:5173/bmdashboard/timelog
  7. Select a project from the dropdown
  8. Verify the following:
    • Time Log Summary Table appears at the top showing completed time logs (if any exist)
    • Member names and roles are displayed correctly in the summary (not "Unknown User")
    • All project members are displayed in cards below the summary
    • Start timers for different members - they should all start from 00:00:00 (not from old values)
    • Pause a timer - it should show the exact time it was paused at (not jump to old value)
    • Resume a paused timer - it should continue from where it was paused (not jump to large old value)
    • Stop a timer - the final time should remain visible on the card (not reset to 00:00:00)
    • Summary table updates automatically after stopping a timer (within ~1 second) without page refresh
    • New completed log appears in the summary table with correct member info, times, and duration

Screenshots or videos of changes:

image image image image

@netlify
Copy link

netlify bot commented Jan 8, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 963418c
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/695f392b3b56a400080c96b5
😎 Deploy Preview https://deploy-preview-4685--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 8, 2026

@vamsidharpanithi vamsidharpanithi changed the title fix: add time log summary component and fix timer display issues Vamsidhar - add time log summary component and fix timer display issues Jan 8, 2026
@vamsidharpanithi vamsidharpanithi added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jan 8, 2026
Copy link
Contributor

@sundarmachani sundarmachani left a comment

Choose a reason for hiding this comment

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

I’ve reviewed this PR and tested the changes locally. Everything is working as expected.
• The Time Log Summary component renders correctly and displays completed logs with accurate member names, roles, start/stop times, and durations.
• Timer behavior issues are resolved — timers now start from 00:00:00, preserve paused/resumed states correctly, and clear properly on stop.
• No regressions observed on the Member Group Check-In page.
• I also verified the changes using before and after screenshots, and the improvements are clearly reflected.

Overall, the implementation looks solid and aligns well with the related backend changes.

Before:
Screenshot 2026-01-10 at 7 10 06 AM
Screenshot 2026-01-10 at 7 21 36 AM

After:
Screenshot 2026-01-10 at 7 19 22 AM
Screenshot 2026-01-10 at 7 19 29 AM

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.

3 participants