Skip to content

Health Indicator #1253

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.

Sign up for GitHub

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

Jump to bottom
Open
dr-bizz wants to merge 126 commits into
base: main
Choose a base branch
from
Open

Health Indicator #1253

dr-bizz wants to merge 126 commits into from

Conversation

@dr-bizz
Copy link
Contributor

@dr-bizz dr-bizz commented Jan 9, 2025

Description

Please explain a bullet-point summary of the changes.
List any PRs that this PR is dependent on and any Jira tickets that this PR is related to.

Checklist:

  • I have given my PR a title with the format "MPDX-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels. (Add the label "On Staging" to get the branch automatically merged into staging.)
  • I have requested a review from another person on the project

canac and others added 30 commits December 5, 2024 11:12
@canac
Add config file for GraphQL language server
3d157e1
@canac
Extract typeguard into lib
3c6d694
@canac
Refactor monthYearFormat to accept DateTime instances
d6ddc19
@canac
Remove unused prop
f626902
@canac
Add button to reset goal to calculated goal
6134977
@canac
Add tests
8bae54b
@canac
Submit the form immediately
59cdef8
@canac
Create reusable LegendReferenceLine component
c34e68e
@canac
Create reusable BarChartSkeleton component
c202dcf
@canac
Unconditionally mock window.ResizeObserver
788db86 
window.ResizeObserver is undefined, so there is no reason to mock it
sometimes and restore it afterwards.
@canac
Create HealthIndicatorGraph component
71868d1
@canac
Remove Hi suffix from field names
4537890
@canac
Add graph colors to theme
31a2a39
@dr-bizz
Create the bare bones of the Health Indicator widget
fd88993
@dr-bizz
Add the widget to the dashboard
4fd1765
@dr-bizz
Switching from makeStyles() to styled() components so we can pass in …
00f4250 
…parameters to change the height of the process bar.
@dr-bizz
Making dashboard responsive with new widget
17d4511
@dr-bizz
Hide HI widget when no data
22ca6ca
@dr-bizz
Adding tests
b4015ca
@dr-bizz
All locales will now be able to view the progress bar as we're no lon…
33c9b3d 
…ger using the localePercentage function
@dr-bizz
Adding Tootltip about where the goal was created on monthlyGoal
d32225f
@dr-bizz
Add percentage to overall score
8764489
@dr-bizz
Merge pull request #1229 from CruGlobal/add-hi-graph-to-dashboard-clean
3967d8e 
MPDX-8478 - Add MPD-HI Card to Dashboard
@canac
Make reset button change the goal to null
35fe64e
@canac
Show the machine calculated goal currency
0463bfa 
Previously the account currency was being used instead of the currency
of the machine calculated goal
@canac
Merge pull request #1217 from CruGlobal/8481-hi-report-graph
5146750 
[MPDX-8481] Health Indicator graph component
@canac
Make currency nullable
580e7b1
@canac
Load all months and use the most recent one
6eb69ac
@canac
Refactor nested ternary
259374e
@canac
Merge pull request #1218 from CruGlobal/8480-hi-preferences
e9e0beb 
[MPDX-8480] Health Indicator preferences page
canac added 30 commits April 15, 2025 10:05
@canac
Warn if the goal is below the machine-calculated goal
091612a
@canac
Add low goal, old goal, and machine-calculated goal annotations to da…
b348d96 
…shboard
@canac
Extract HI logic into reusable module
4f01759
@canac
Use getHealthIndicatorInfo in AccountLists
473ee43
@canac
Use getHealthIndicatorInfo in MonthlyGoalAccordion
09777b5
@canac
Add low goal warning to My Accounts grid
100da01
@canac
Adjust annotations based on new stakeholder mockups
b0fee78
@canac
Add success/warning/error colors to the theme
70b8d64
@canac
Use getHealthIndicatorInfo in graphData
e5651a9
@canac
Remove unused mismatchedCurrencies property
8ff83fd
@canac
Move the annotation next to the goal when there isn't HI data
2b30e9b
@canac
Merge pull request #1284 from CruGlobal/8566-goal-warning
7aff140 
[MPDX-8566] Preference goal warnings
@canac
Merge branch 'health-indicator' into 8565-preference-goal-date
fc1024e
@canac
Load currency from AccountList instead of AccountListSettings
616e6ce 
AccountList.currency is given a default value on the server, so it will
always be set, while AccountListSettings.currency is nullable. We need
to load the currency the same way as we do in the rest of the
application so that it is available for the health indicator
calculations.
@canac
Merge pull request #1283 from CruGlobal/8565-preference-goal-date
89e4489 
[MPDX-8565] Indicate when the preference goal was last updated
@canac
Stop loading all HI periods when only the latest is needed
9d74ca0
@canac
Use getHealthIndicatorInfo in DonationHistories
b4efd8d
@canac
Replace machine-calculated labels with NetSuite-calculated
5b7b0e0
@canac
Replace "machine calculated" with "machine-calculated" in test names
d0afb66
@canac
Merge pull request #1292 from CruGlobal/8567-optimize-queries
c05b446 
[MPDX-8567] Optimize Health Indicator queries
@canac
Merge pull request #1299 from CruGlobal/8569-netsuite-calculated
4b5379f 
[MPDX-8579] Replace machine-calculated labels with NetSuite-calculated
@canac
Merge branch 'main' into health-indicator
a4a1059
@canac
Make the goal line dashed for users without HI data
77a0ac5
@canac
Update health indicator data missing message
ad109f5
@canac
Merge pull request #1316 from CruGlobal/8569-goal-line-dashed
fe5508d 
[MPDX-8569] Dashed donations goal line
@canac
Merge pull request #1317 from CruGlobal/8568-missing-data-message
8a9dd55 
[MPDX-8568] Improve Health Indicator missing data message
@canac
Remove missing indicators from the weighted average
11ea806
@canac
Add the detailed MPD Health formula description
b8d3e42
@canac
Create constants for weights
4c5c41f
@canac
Merge pull request #1320 from CruGlobal/8582-missing-indicators
35142f3 
[MPDX-8582] Adjust calculations when indicators are absent
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Preview Environment Add this label to create an Amplify Preview

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dr-bizz @canac