-
Notifications
You must be signed in to change notification settings - Fork 347
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
feat: Temperature preference stored in local storage #5589
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Add stricter types and refactor to move
getTemperaturePreference
to utils. - Why have you not implemented code to use this preference in places where temperatures are shown?
Kudos, SonarCloud Quality Gate passed!
|
LGTM |
This reverts commit 556875b.
WHAT
🤖 Generated by Copilot at f4b8d2a
The pull request adds a temperature unit preference feature to the user profile. It introduces a new
TemperatureSelector
component that handles the user's choice and uses it in theUserProfile
component.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at f4b8d2a
TemperatureSelector
to let the user choose between Celsius and Fahrenheit units for temperature (link)TemperatureSelector
component in theUserProfile
component, which shows and edits the user's profile information (link, link)TemperatureSelector
component to match the form layout and background color in theUserProfile
component (link)