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

Changed weekly to average weekly for working hours #6228

Merged
merged 8 commits into from
Sep 6, 2023

Conversation

kshitijv256
Copy link
Contributor

@kshitijv256 kshitijv256 commented Sep 5, 2023

WHAT

🤖 Generated by Copilot at 298e59d

The pull request updates the user interface of the ManageUsers.tsx and UserProfile.tsx components to use the term Average weekly working hours instead of weekly working hours for the working hours field. This change aligns with the new requirement of showing the average working hours of users.

Proposed Changes

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

@kshitijv256 kshitijv256 requested a review from a team September 5, 2023 17:55
@kshitijv256 kshitijv256 requested a review from a team as a code owner September 5, 2023 17:55
@vercel
Copy link

vercel bot commented Sep 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
care-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 5, 2023 5:56pm

@netlify
Copy link

netlify bot commented Sep 5, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 298e59d
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/64f76b8eacec8400083d6a0e
😎 Deploy Preview https://deploy-preview-6228--care-egov-staging.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.

@nihal467
Copy link
Member

nihal467 commented Sep 6, 2023

LGTM

@nihal467 nihal467 added the tested label Sep 6, 2023
@khavinshankar khavinshankar merged commit 995f4d7 into coronasafe:develop Sep 6, 2023
32 of 33 checks passed
ShivamJhaa pushed a commit to ShivamJhaa/care_fe that referenced this pull request Sep 6, 2023
* Added padding to count block on patients page

* fixed date format in asset manage page

* Merged configure facility and configure health facility in one page

* removed commented code

* changed weekly to average weekly for working hours
khavinshankar pushed a commit that referenced this pull request Sep 7, 2023
…6106)

* lints

* Added test for resource and migrated it to POM

* trying a fix

* nits

* backend change

* Reverted useless changes

* fix test

* Fixed failing test

* Added api verification

* Diagnosis: Adds field Principal Diagnosis (#6218)

* adds field principle diagnosis

* show in consultation details

* fix typo

* fix cypress

* try adding wait

* fix tests

* open pdf file preview in new tab (#6226)

* Changed weekly to average weekly for working hours (#6228)

* Added padding to count block on patients page

* fixed date format in asset manage page

* Merged configure facility and configure health facility in one page

* removed commented code

* changed weekly to average weekly for working hours

* Resolved comments

---------

Co-authored-by: Mohammed Nihal <57055998+nihal467@users.noreply.github.com>
Co-authored-by: Rithvik Nishad <mail@rithviknishad.dev>
Co-authored-by: Ashesh <3626859+Ashesh3@users.noreply.github.com>
Co-authored-by: Kshitij Verma <101321276+kshitijv256@users.noreply.github.com>
@kshitijv256 kshitijv256 deleted the awwh branch September 8, 2023 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Weekly working hours (Add average)
3 participants