-
Notifications
You must be signed in to change notification settings - Fork 9
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
DNM: Integration form engine 2 #2273
Draft
ryguyk
wants to merge
21
commits into
main
Choose a base branch
from
integration-form-engine-2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 16 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
01edd06
VATEAM-87714: Add KISS configuration for Digital Forms (#2213)
derekhouck fd22547
VATEAM-88634: Create a normalization layer for Digital Forms (#2215)
derekhouck 49ce1af
Add OMB info to Digital Form GraphQL query
derekhouck 0cec642
Normalize OMB info
derekhouck 4d7dc20
Remove Date object from formatDate
derekhouck 52edb57
VATEAM-90733: Normalize OMB info fields (#2252)
derekhouck b64c733
Add Identification Information fragment
derekhouck 62fdaab
Import identificationInformation fragment into digitalForm fragment
derekhouck ee1a69f
Extract Identification Information fields
derekhouck 9a6014e
Fix imports
derekhouck 66f5048
VATEAM-87714: Add KISS configuration for Digital Forms (#2213)
derekhouck e3d84e6
VATEAM-88634: Create a normalization layer for Digital Forms (#2215)
derekhouck a314931
Merge branch 'main' into integration-form-engine-2
derekhouck 3633329
Merge branch '90733-normalize-omb-info-section' into integration-form…
derekhouck 628f6cb
Merge branch '91741-normalize-identification-information' into integr…
derekhouck 0166a12
VATEAM-92265: Normalize Address Digital Form pattern output (#2301)
derekhouck 02d5d12
VATEAM-94290: Remove "Digital Form:" from content-build output (#2304)
derekhouck 1d80dde
Merge branch 'main' into integration-form-engine-2
derekhouck a0310be
VATEAM-93401: Normalize Phone and Email Digital Form pattern output (…
derekhouck 00433c4
VATEAM-94709: Normalize Your Personal Information chapter (#2338)
derekhouck 56aee27
VATEAM-93614: Normalize List and Loop pattern output (#2350)
derekhouck File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
14 changes: 14 additions & 0 deletions
14
src/site/stages/build/drupal/static-data-files/digitalForm/fragments/address.graphql.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
/* | ||
* | ||
* The "Address" Digital Form pattern. | ||
* | ||
* Pattern documentation: | ||
* https://design.va.gov/patterns/ask-users-for/addresses | ||
* | ||
*/ | ||
module.exports = ` | ||
fragment address on ParagraphDigitalFormAddress { | ||
fieldTitle | ||
fieldMilitaryAddressCheckbox | ||
} | ||
`; |
13 changes: 13 additions & 0 deletions
13
.../stages/build/drupal/static-data-files/digitalForm/fragments/address.graphql.unit.spec.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
/* eslint-disable @department-of-veterans-affairs/axe-check-required */ | ||
|
||
import { expect } from 'chai'; | ||
import address from './address.graphql'; | ||
|
||
describe('address fragment', () => { | ||
it('includes fieldTitle', () => { | ||
expect(address).to.have.string('fieldTitle'); | ||
}); | ||
it('includes fieldMilitaryAddressCheckbox', () => { | ||
expect(address).to.have.string('fieldMilitaryAddressCheckbox'); | ||
}); | ||
}); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
14 changes: 14 additions & 0 deletions
14
...build/drupal/static-data-files/digitalForm/fragments/identificationInformation.graphql.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
/* | ||
* | ||
* The "Identification Information" Digital Form pattern. | ||
* | ||
* Pattern documentation: | ||
* https://design.va.gov/patterns/ask-users-for/social-security-number | ||
* | ||
*/ | ||
module.exports = ` | ||
fragment identificationInformation on ParagraphDigitalFormIdentificationInfo { | ||
fieldTitle | ||
fieldIncludeVeteranSService | ||
} | ||
`; |
15 changes: 15 additions & 0 deletions
15
...al/static-data-files/digitalForm/fragments/identificationInformation.graphql.unit.spec.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
/* eslint-disable @department-of-veterans-affairs/axe-check-required */ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ESLint disabled here |
||
|
||
import { expect } from 'chai'; | ||
import identificationInformation from './identificationInformation.graphql'; | ||
|
||
describe('identificationInformation fragment', () => { | ||
it('includes fieldTitle', () => { | ||
expect(identificationInformation).to.have.string('fieldTitle'); | ||
}); | ||
it('includes fieldIncludeVeteranSService', () => { | ||
expect(identificationInformation).to.have.string( | ||
'fieldIncludeVeteranSService', | ||
); | ||
}); | ||
}); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ESLint disabled here