Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(feat) O3-3748: Add data-openmrs-role data attributes to DOM elements #1953

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

Vijaykv5
Copy link
Contributor

@Vijaykv5 Vijaykv5 commented Aug 6, 2024

Requirements

  • This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes a conventional commit label. See existing PR titles for inspiration.
  • My work conforms to the OpenMRS 3.0 Styleguide and design documentation.
  • My work includes tests or is validated by existing tests.

Summary

This PR defines a proper way for selectors accesability. By having this, we can query element using data-openmrs-role in the dom.

Screenshots

Related Issue

O3-3748

Other

As per this #comment

cc : @jayasanka-sack

@denniskigen denniskigen changed the title (enhnc) O3-3748 : Added data-openmrs-role property to buttons and forms (feat) O3-3748: Add data-openmrs-role data attributes to DOM elements Aug 6, 2024
@denniskigen
Copy link
Member

Thanks!

@denniskigen denniskigen merged commit 01e4c39 into openmrs:main Aug 7, 2024
6 checks passed
@Vijaykv5 Vijaykv5 deleted the enhnc/O3-3748 branch August 7, 2024 18:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants