Skip to content

chore: update submission name pattern #174

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

Merged
merged 1 commit into from
May 27, 2025

Conversation

timarney
Copy link
Member

Summary | Résumé

Update to handle legacy + new submission name based on

cds-snc/forms-terraform#1032

Copy link
Contributor

@craigzour craigzour left a comment

Choose a reason for hiding this comment

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

Just missing a test in router.test.ts.

Adding a new test object after line 97 with the new submission name version should be enough :)

@timarney
Copy link
Member Author

Added the test obj

@timarney timarney requested a review from craigzour May 22, 2025 14:31
Copy link
Contributor

@craigzour craigzour left a comment

Choose a reason for hiding this comment

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

Looks good! Let's wait and see what is the priority on getting this to production (context: https://gcdigital.slack.com/archives/C01GFD150MC/p1747921792463709?thread_ts=1747830335.978189&cid=C01GFD150MC)

@craigzour craigzour enabled auto-merge (squash) May 27, 2025 14:23
@craigzour craigzour force-pushed the update-submission-name-pattern branch from c4c3245 to e7c50fc Compare May 27, 2025 14:31
@craigzour craigzour merged commit fd2c0c7 into main May 27, 2025
4 checks passed
@craigzour craigzour deleted the update-submission-name-pattern branch May 27, 2025 14:31
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