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

Minor content changes for 2023 #139480

Merged
merged 3 commits into from
Oct 11, 2023
Merged

Minor content changes for 2023 #139480

merged 3 commits into from
Oct 11, 2023

Conversation

benmartin-coforma
Copy link
Contributor

Description

  1. Add calculation explanation to Section 2, Part 1: "Number of Children enrolled in CHIP"
  2. Remove past years from Section 2, Part 2. It only needs to go back 5 years, so for 2023, going back to 2018 is sufficient.
  3. Add "Part 1" and "Part 2" headers to Section 3I. This is tricky and maybe not the best solution, but Part.js line 55 prevents these headers from displaying unless the part has a title. I don't have any text to add for the title but I can add an invisible truthy string: U+00A0, the non-breaking space. This causes the headers to render as Part 1: and Part 2:. If the trailing colon is not desirable, then this request will require a broader change.
  4. Change repeatable question text in Section 3I to "Does the state have additional HSIs to report?". This required a schema change to optionally pass in this text. I don't believe it will affect any other questions.
  5. Change repeatable button text in Section 3I to "Add Another HSI Program". The requested text was "Add Another HSI", but the JSON already had "typeLabel": "HSI Program", and an apparent bug: it seems like it was always intended for this text to pass through to the button text. Additionally, it seems like it was always intended to pass this through to the "do you have another" question text, as well as the "delete last" button text. I've "fixed" it here, but THIS MAY NOT BE A GOOD IDEA, as it will affect all repeatables throughout the report. This includes all six repeatables in Section 4, which will now ask "Do you have another Goal in this list?" with button text "Add another Goal".
  6. Hide the "Optional" text under "Does the state have additional HSIs to report?". This was another schema change, to optionally pass a hideOptionalHint flag through the question.

Related ticket(s)

MDCT-3035


How to test

  1. Log in to CARTS as an admin
  2. Generate the 2023 forms
  3. Log in to CARTS as a state user
  4. Click through the report to observe the changes

Important updates

n/a


Author checklist

  • I have performed a self-review of my code
  • I have added thorough tests, if necessary
  • I have updated relevant documentation, if necessary

@BearHanded
Copy link
Contributor

BearHanded commented Oct 6, 2023

Add "Part 1" and "Part 2" headers to Section 3I. ... If the trailing colon is not desirable, then this request will require a broader change.

This one I could go either way on, if only due to timing. My guess is the trailing colon will be fine. It's worth running by David, though.

Copy link
Contributor

@gmrabian gmrabian left a comment

Choose a reason for hiding this comment

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

We want to hide the 'optional' hint, right?
Otherwise changes look good. I verified the trailing colon and non-breaking space don't have any issues when read with VoiceOver (It just says "Part 1" and moves on)

services/database/data/seed/seed-section-base-2023.json Outdated Show resolved Hide resolved
@codeclimate
Copy link

codeclimate bot commented Oct 11, 2023

Code Climate has analyzed commit a5addc8 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 61.9% (0.2% change).

View more on Code Climate.

@benmartin-coforma benmartin-coforma marked this pull request as ready for review October 11, 2023 13:56
@benmartin-coforma benmartin-coforma enabled auto-merge (squash) October 11, 2023 13:56
@benmartin-coforma benmartin-coforma merged commit caa5805 into main Oct 11, 2023
14 checks passed
@benmartin-coforma benmartin-coforma deleted the content-changes-3035 branch October 11, 2023 14:35
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.

3 participants