[Prod] Node update, edit goals in place, accessibility and bug fixes, initial TR changes #3141

thewatermethod · 2025-11-24T17:59:27Z

Description of change

Edit goals in place on the AR.
Comm Log Next Steps date fields were showing "enter a valid date" errors even though they were not required
Fixed the "Add new resource" button so it shows an error when trying to add a new resource when the current one was blank
[TTAHUB-4620] Upgrade NodeJS to 22.20.0
Show "Add new goals" button when goals available after deletion
Hide the create communication log button if the user doesn't have the correct permissions.
Adjust hsesUserId in processData to resolve login issues in non-production environments
Address issues with google tag manager data bridge
Training report updates, in progress

How to test

Ensure goals can be edited in placed while working on the AR goals and objectives page.
A user who doesn't have write permission in their home region should not be able to create a comm log.

Issue(s)

Checklists

Every PR

Before merge to main

OHS demo complete
Ready to create production PR

Production Deploy

PR created as Draft
Staging smoke test completed
PR transitioned to Open
Reviewer added (after transitioning to Open to ensure Slack notifications trigger)
- Sequence: Draft PR → Smoke test → Open PR → Add reviewer
- Confirm that Slack notification was sent after reviewer was added

After merge/deploy

Update JIRA ticket status

…revious resource is blank.

…o better support required date fields

…rt-TTADP into 4165-ar-new-res-err

…e permissions

TTAHUB-4165 - New AR Resource Error when Prev Res is blank

TTAHUB-4178 - Prevent errors for blank dates on Next Steps

* upgrade node to 22.20.0 * deps * fix image format * go back * cheerio * skiplibcheck * no esm * esm * frontend

* [TTAHUB-4190] Update session participants form (#3048) * Surface adjustments to session form * Add tooltip * Update end-to-end test * More specific e2e selector * [TTAHUB-4189] Adjust objective trainers field within the session summary to use a dynamic list (#3046) * Add a new seeder and option to event summary audience * v2.2.15 * template updates and test fixes * Add fetcher test * add new service test * Add new handlers * fix e2e test * increase stacktrace size to 50 (#3044) * [TTAHUB-4569] Propagate errors during daily import job (#3047) * propate cron errors * test * test * test * cleanup * Fix bug and test (#3057) --------- Co-authored-by: Tom Meagher <[email protected]>

* Add new hook and test * Update preview target

#3124) * Hook-ify useUrlFilters to not double tap the history API * Fix useFilters test * Remove goal dashboard test, fix comm log dashboard test * Finish hookifying, fix some tests * Add console.log for QA verification * Remove console.log

[TTAHUB-3087] Edit goals in place

…deletion (#3138) * TTAHUB-4421 Do not show add new goals button when all of the standard goals have been used * Improve code coverage * Add try/catch to tests to debug * Remove try/catch * Further test adjustments * Fix missing button on goal delete

…-log

…t-in-comm-log Hide create communication log button if use doesn't have permissions

AdamAdHocTeam and others added 30 commits November 7, 2025 15:42

edit goals in place

9b76d3f

ensure buttons appear below goal being edited

ea20754

fix tests

7db61ca

fix save draft keeps edit in place

db3710a

Adding "not required" to completeDate validation checking

28a4bb3

fix tests

df1ddc2

fix some page state issues with save draft

2d74379

merge main

0e22f69

fix test

e4b6c7f

add coverage

84a70af

lint

bea1b68

see if this fixes laurens bug

f598d99

audit

e88935a

test coverage

a27134c

button margin

40deebd

hide goal picker if loading

c1a7aea

coverage

1a9832f

fix lint

e445bda

also hide the save buttons during save

fbbb412

Some initial NextStepsRepeater tests

d0aead1

Yarn audit update

9a70bcc

Merge branch 'main' into 4178-comm-log-next-steps-dates

6889e64

roll back unwanted ux change

43e6da8

Merge branch 'main' into 4178-comm-log-next-steps-dates

b536f1b

Adding an error when trying to add a new resource to an AR when the p…

53651d0

…revious resource is blank.

Merge branch 'main' into 4165-ar-new-res-err

67e8666

Merge branch 'main' into 4178-comm-log-next-steps-dates

8591b35

Improvement to NextStepsRepeater showCompleteDateError calculations t…

ee76327

…o better support required date fields

Removing console logs

ef75536

Merge branch '4165-ar-new-res-err' of https://github.com/HHS/Head-Sta…

93ca145

…rt-TTADP into 4165-ar-new-res-err

AdamAdHocTeam and others added 22 commits November 18, 2025 15:40

fixes

cbaca3a

merge main

a00f1d8

lint

85cdbc4

Merge branch 'main' into al-ttahub-3087-goal-edit-in-place-fix

71ef4cc

Adding a check onBlur for clearing the showNoResourcesError state

403c3c8

Merge branch 'main' into 4178-comm-log-next-steps-dates

ca6c71b

hide the comm log create button if users home region doesnt have writ…

4f8c71d

…e permissions

Merge branch 'main' into 4165-ar-new-res-err

9a08e76

Merge branch 'main' into 4178-comm-log-next-steps-dates

84018b1

Merge pull request #3129 from HHS/4165-ar-new-res-err

d57381e

TTAHUB-4165 - New AR Resource Error when Prev Res is blank

Merge branch 'main' into 4178-comm-log-next-steps-dates

181f68a

Merge pull request #3118 from HHS/4178-comm-log-next-steps-dates

7609478

TTAHUB-4178 - Prevent errors for blank dates on Next Steps

[TTAHUB-4620] Upgrade node to 22.20.0 (#3121)

1ce7e8f

* upgrade node to 22.20.0 * deps * fix image format * go back * cheerio * skiplibcheck * no esm * esm * frontend

Merge branch 'main' into al-ttahub-3087-goal-edit-in-place-fix

5491804

[TTAHUB-4617] Add hook to populate GTM content variable (#3130)

f23affe

* Add new hook and test * Update preview target

Merge branch 'main' into al-ttahub-3087-goal-edit-in-place-fix

0d1ff19

Check for onClick before invoking (#3137)

9499e0a

Merge branch 'main' into al-ttahub-3087-goal-edit-in-place-fix

3ea1263

Merge pull request #3109 from HHS/al-ttahub-3087-goal-edit-in-place-fix

af8da01

[TTAHUB-3087] Edit goals in place

thewatermethod changed the title ~~[Prod]~~ [Prod] Node update, edit goals in place, accessibility and bug fixes, initial TR changes Nov 24, 2025

AdamAdHocTeam and others added 3 commits November 24, 2025 15:09

Merge branch 'main' into al-support-case-error-entering-event-in-comm…

96e5dfa

…-log

Merge pull request #3136 from HHS/al-support-case-error-entering-even…

fb5b71a

…t-in-comm-log Hide create communication log button if use doesn't have permissions

Adjust hsesUserId for Andrew (#3142)

44003b7

thewatermethod marked this pull request as ready for review November 25, 2025 14:04

thewatermethod requested a review from elainaparrish November 25, 2025 14:04

elainaparrish approved these changes Nov 25, 2025

View reviewed changes

elainaparrish merged commit d6d389c into production Nov 25, 2025
44 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Prod] Node update, edit goals in place, accessibility and bug fixes, initial TR changes #3141

[Prod] Node update, edit goals in place, accessibility and bug fixes, initial TR changes #3141

Uh oh!

thewatermethod commented Nov 24, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

[Prod] Node update, edit goals in place, accessibility and bug fixes, initial TR changes #3141

[Prod] Node update, edit goals in place, accessibility and bug fixes, initial TR changes #3141

Uh oh!

Conversation

thewatermethod commented Nov 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of change

How to test

Issue(s)

Checklists

Every PR

Before merge to main

Production Deploy

After merge/deploy

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

thewatermethod commented Nov 24, 2025 •

edited

Loading