-
Notifications
You must be signed in to change notification settings - Fork 1
[MPDX-8566] Preference goal warnings #1284
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
Conversation
Preview branch generated at https://8566-goal-warning.d3dytjb8adxkk5.amplifyapp.com |
Bundle sizes [mpdx-react]Compared against 1d1fd1a
|
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.
Great work on this! I left a few comments but it's looking awesome
ba14203
to
dbe7bc4
Compare
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.
It's looking great! I've left a few comments, but I think it's looking good. I'll approve to unblock you
adadb8b
to
2b30e9b
Compare
Description
The goal is to implement a UI similar to this mock-up from stakeholders. Note that we are considering language change to something like "system-generated support goal" but that is still being discussed.
MPDX-8566
Testing
src/lib/healthIndicator.ts
changeDateTime.now().minus({ year: 1 })
toDateTime.now()
Checklist: