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

Add Download XDR Cert shared component #155

Merged
merged 4 commits into from
Oct 15, 2024

Conversation

brooke-ritchey
Copy link
Contributor

No description provided.

@drbgfc drbgfc self-assigned this Oct 10, 2024
@drbgfc drbgfc requested a review from akanuri9 October 10, 2024 16:02
@drbgfc drbgfc assigned akanuri9 and unassigned drbgfc Oct 10, 2024
Copy link
Contributor

@akanuri9 akanuri9 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, 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') {
Copy link
Contributor

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' ||
Copy link
Contributor

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.

@akanuri9 akanuri9 merged commit c4faba7 into dev Oct 15, 2024
3 checks passed
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.

3 participants