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.

Sign up for GitHub

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

Jump to bottom

Add 10-10EZR schema #794

Merged
merged 50 commits into from
Oct 18, 2023
Merged

Add 10-10EZR schema #794

merged 50 commits into from
Oct 18, 2023

Conversation

lihanli
Copy link
Contributor

@lihanli lihanli commented Aug 22, 2023 •
edited
Loading

New schema

new schema for the 10-10EZR form
department-of-veterans-affairs/va.gov-team#63591

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.

lihanli added 30 commits August 14, 2023 18:46
@lihanli
put hcafullname into definitions
c67cdb9
@lihanli
working on ezr schema
6283df4
@lihanli
change form name
3966e71
@lihanli
add hca dependent definition
2189909
@lihanli
move hcadependent to after ssn def
ff1fa16
@lihanli
add hca monetary value definition
ff9ba01
@lihanli
make dependents a definition
f23825e
@lihanli
remove unused field
f3eb66e
@lihanli
remove optional field
9d101c1
@lihanli
use format date in hca dependents def
0a59671
@lihanli
build
eb11ec0
@lihanli
remove unused include
6371f87
@lihanli
put hca address in definition
3166aa5
@lihanli
add name fields
48e1642
@lihanli
add ssn
1365cf9
@lihanli
add gender
6e1e26c
@lihanli
add sigi genders
8220e77
@lihanli
add dob
6ac34bf
@lihanli
add phone nums
62dd990
@lihanli
add address fields
85c316d
@lihanli
add email
98227a9
@lihanli
add marital status
5271b2a
@lihanli
is covered by insurance bool
f1deb0b
@lihanli
add insurance provider def
2f5da80
@lihanli
add providers to 1010ezr
24084f6
@lihanli
medicaid eligible
633c7ef
@lihanli
medicare bool
beabe20
@lihanli
medicare fields
5932ccb
@lihanli
build
6222beb
@lihanli
add 1010ezr spec
8f48be8
@JoshingYou1 JoshingYou1 requested a review from longmd October 5, 2023 18:48
@JoshingYou1 JoshingYou1 marked this pull request as ready for review October 5, 2023 18:49
@JoshingYou1 JoshingYou1 requested review from a team as code owners October 5, 2023 18:49
@RachalCassity
Copy link
Member

@lihanli Is this PR still relevant or can this PR be closed?

@lihanli
Copy link
Contributor Author

lihanli commented Oct 17, 2023 •
edited
Loading

@RachalCassity @JoshingYou1 has taken over this PR, but we still want to merge this

lihanli
lihanli commented Oct 17, 2023
View reviewed changes
src/common/definitions.js Outdated Show resolved Hide resolved
@lihanli
Copy link
Contributor Author

lihanli commented Oct 18, 2023

@JoshingYou1 you need to run yarn run build and commit

lihanli
lihanli commented Oct 18, 2023
View reviewed changes
dist/10-10EZ-schema.json Outdated Show resolved Hide resolved
lihanli
lihanli commented Oct 18, 2023
View reviewed changes
Copy link
Contributor Author

@lihanli lihanli left a comment

Choose a reason for hiding this comment

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

LGTM

longmd
longmd approved these changes Oct 18, 2023
View reviewed changes
Copy link
Contributor

@longmd longmd left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@platform-support-slack-integration platform-support-slack-integration bot mentioned this pull request Oct 18, 2023
Closed
@JoshingYou1 JoshingYou1 merged commit c2c0bf4 into master Oct 18, 2023
5 checks passed
@JoshingYou1 JoshingYou1 deleted the fd-form branch October 18, 2023 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JoshingYou1 JoshingYou1 JoshingYou1 left review comments

@RachalCassity RachalCassity RachalCassity approved these changes

@longmd longmd longmd approved these changes

@jadon1979 jadon1979 jadon1979 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lihanli @RachalCassity @jadon1979 @longmd @JoshingYou1