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

feat: [CGC-65] Automatic time interval selection #83

Merged
merged 11 commits into from
Sep 20, 2024

Conversation

edvinstava
Copy link
Collaborator

@edvinstava edvinstava commented Jun 4, 2024

This PR enhances the functionality of the growth chart plugin by selecting the appropriate time interval for the selected child's age by default.

Note: This PR makes the growth chart require a date of birth. It will therefore not work without a tracked entity.

@edvinstava edvinstava requested review from petterav and henrikmv June 4, 2024 10:30
@petterav petterav removed request for henrikmv and petterav June 6, 2024 14:28
@edvinstava edvinstava requested review from henrikmv and petterav June 10, 2024 07:54
Copy link
Collaborator

@petterav petterav left a comment

Choose a reason for hiding this comment

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

Weight for age shows 0-2 years even though the child is 3 years. This is because the keys are different for this particular indicator. Not a big issue, because this feature is just to help improve the efficieny, but it is also possible to create a scenario for this particular indicator.

Well done Edvin. The code works extremely good! Look at the comments and i will review again!

src/utils/Hooks/Calculations/useAppropriateChartData.ts Outdated Show resolved Hide resolved
src/utils/Hooks/Calculations/useAppropriateChartData.ts Outdated Show resolved Hide resolved
src/utils/Hooks/Calculations/useAppropriateChartData.ts Outdated Show resolved Hide resolved
src/utils/Hooks/Calculations/useAppropriateChartData.ts Outdated Show resolved Hide resolved
@edvinstava edvinstava requested a review from petterav June 19, 2024 11:23
…omaticTimeIntervalSelection

# Conflicts:
#	i18n/en.pot
#	src/components/GrowthChart/GrowthChart.tsx
Copy link
Collaborator

@petterav petterav left a comment

Choose a reason for hiding this comment

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

DefaultIndicatorError, can you change from 'is not a valid.' To 'is not valid'

Looks very good Edvin, make the few changes commented and i will approve the PR!

@edvinstava edvinstava requested a review from petterav August 6, 2024 12:44
Copy link
Collaborator

@petterav petterav left a comment

Choose a reason for hiding this comment

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

LGTM! Look at the one comment before merging with master, well done Edvin

src/Plugin.tsx Outdated Show resolved Hide resolved
@edvinstava edvinstava merged commit ee37c00 into master Sep 20, 2024
4 checks passed
@edvinstava edvinstava deleted the eas/feat/CGC-65_AutomaticTimeIntervalSelection branch September 20, 2024 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants