-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add Download XDR Cert shared component #155
Conversation
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.
Looks good, only change is related to H2 to add for other categories where there is XDR.
@@ -46,6 +48,11 @@ const H2Component = () => { | |||
const criteriaList = h2CriteriaList.filter((c) => c.selectOption?.includes(selectedOption[0].selectOption)) | |||
setCriteriaOptions(criteriaList) | |||
setSelectedCriteria('') | |||
if (event.target.value === 'Send conversion XDR' || event.target.value === 'Receive conversion XDR') { |
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.
I believe there few others categories in H2 which have XDR related test cases, you can check here https://ett.healthit.gov/ett/#/certification/certh2 as H2 implementation is not yet completed.
} | ||
|
||
const handleCriteriaChange = (event: SelectChangeEvent) => { | ||
setSelectedCriteria(event.target.value as string) | ||
if ( | ||
event.target.value === 'Criteria (i) XDR Validator' || |
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.
Instead of adding each criteria, may be you can add a condition such as where the value contains XDR you add the button that way you dont have to worry for any criteria is missed.
No description provided.