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

Rename form 28-8832 to 27-8832 pt 2. #859

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Conversation

zack
Copy link
Contributor

@zack zack commented Mar 13, 2024

Forgot to generate dist/version.json

Pt. 1: #858

Issue: department-of-veterans-affairs/va.gov-team#78359

Pull Requests to update the schema in related repositories

After you've merged your changes to vets-json-schema you'll need to make PR's to vets-website and vets-api. Please link them here.

@zack zack requested review from a team as code owners March 13, 2024 19:30
@zack
Copy link
Contributor Author

zack commented Mar 13, 2024

Sorry @LindseySaari :)

@zack zack merged commit 81fe186 into master Mar 13, 2024
5 checks passed
@zack zack deleted the zy-ch36-form-name-changes-2 branch March 13, 2024 19:43
zack added a commit to department-of-veterans-affairs/vets-website that referenced this pull request Mar 13, 2024
The form use to be 28-8832. Then it was 25-8832. Now it's 27-8832. Lots
of references remain from both of the old names, so this PR updates all
of those. The schema update is already merged across these two PRs
(sorry):

- department-of-veterans-affairs/vets-json-schema#858
- department-of-veterans-affairs/vets-json-schema#859
zack added a commit to department-of-veterans-affairs/vets-website that referenced this pull request Mar 14, 2024
The form use to be 28-8832. Then it was 25-8832. Now it's 27-8832. Lots
of references remain from both of the old names, so this PR updates all
of those. The schema update is already merged across these two PRs
(sorry):

- department-of-veterans-affairs/vets-json-schema#858
- department-of-veterans-affairs/vets-json-schema#859

And the content-build change was made here:

- #28545
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