Required prop in multiselect component sets aria property

[johanna-skylight]

FRONTEND PULL REQUEST

Related Issue

Resolves #6408

Changes Proposed

Add logic so the prop required of the MultiSelect component is set as an aria-required prop.

Additional Information

This fixes the issue for the device edit form as well.

Testing

Verify the aria-required props is set to true.

[BobanL]

looks good to me!

[BobanL]

Actually, I think this only resolves #6404 and not #6408, only the SNOMED code multi selects have aria-requried. The device combo box does not have aria-required.

[johanna-skylight]

Actually, I think this only resolves #6404 and not #6408, only the SNOMED code multi selects have aria-requried. The device combo box does not have aria-required.

I am a little confused... I thought 6408 referred to the multi-select device types (shown below)

And 6404 referred to the multi-select SNOMED (shown below)

Can you show a screenshot of the component that is still missing the aria-required and in which page 🤔

[BobanL]

Actually, I think this only resolves #6404 and not #6408, only the SNOMED code multi selects have aria-requried. The device combo box does not have aria-required.

I am a little confused... I thought 6408 referred to the multi-select device types (shown below)

And 6404 referred to the multi-select SNOMED (shown below)

Can you show a screenshot of the component that is still missing the aria-required and in which page 🤔

Ooooppps, I got confused with #6405 which what I thought #6408 was. This fixes #6404 and #6408 like you said!

[emyl3]

LGTM!

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication