Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sriram TimeLogger for BMdashboard #3299

Open
wants to merge 4 commits into
base: development
Choose a base branch
from

Conversation

sri0606
Copy link
Contributor

@sri0606 sri0606 commented Mar 21, 2025

Description

Implementing "Time Logger" Function for bmdashboard.
http://localhost:3000/bmdashboard/timelog

Related PRS (if any):

Updated PR from #2494

This PR should be tested along with Backend 1283

Main changes explained:

This PR is for implementing the followings:

URL: “/bmdashboard/timelog”

By selecting a specific project, the manager can view the users assigned to the project.
The top bar shows the name of the user. The color may change according to the user’s role. For example, light blue for volunteers, orange for core team, light green for managers and mentors, and red for owner.
The “start”, “stop” or “clear” buttons are the same as the buttons used for time log in the phase one system.
A select dropdown for the manager to monitor. The value here will be the default/pre-select value passed to the time log form to save.

image

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 any user
  5. go to http://localhost:3000/bmdashboard/timelog, you may need to login again.
  6. select a project to test
  7. test start/pause/stop/clear time log for each member

Screenshots or videos of changes:

image

HGN.APP.-.Google.Chrome.2024-07-22.12-52-31.mp4

Note:

Function implemented by now:

  1. retrieve project and member information from server to display member card
  2. Stop/Pause/Stop/Clear time log
  3. Set card color according to member role

Functions to be implemented further

  1. retrieve task name to display on the member's card
    --> may need to think about separate signaling to retrieve task information because of database fetch latency
  2. update time log information to the database
  3. some cosmetic improvement

@sri0606 sri0606 added Medium Priority do not review Do not review or look at code without full context labels Mar 21, 2025
Copy link

netlify bot commented Mar 21, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 121b924
🔍 Latest deploy log https://app.netlify.com/sites/highestgoodnetwork-dev/deploys/67e5d336ed05800008987512
😎 Deploy Preview https://deploy-preview-3299--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 site configuration.

@Amalesh-A
Copy link

I followed your steps, and I was able to replicate the features. I logged as an admin, and was able to observe the following,
Overview
Timer Works!

However, there's a cosmetic improvement that could be made — I noticed that the timer's orientation remains unchanged even when zooming in or out. For example,

Zoom - 1
Zoom - 2
Zoom - 3

Amalesh-A
Amalesh-A previously approved these changes Mar 21, 2025
Copy link

@Amalesh-A Amalesh-A left a comment

Choose a reason for hiding this comment

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

I followed your steps, and I was able to replicate the features. I logged as an admin, and was able to observe the following,
Overview
Timer Works!

However, there's a cosmetic improvement that could be made — I noticed that the timer's orientation remains unchanged even when zooming in or out. For example,

Zoom - 1
Zoom - 2
Zoom - 3

Copy link
Contributor

@vishavk1992 vishavk1992 left a comment

Choose a reason for hiding this comment

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

I have tested this PR and everything working as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not review Do not review or look at code without full context Medium Priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants