-
Notifications
You must be signed in to change notification settings - Fork 370
OUIA audit #4401
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
OUIA audit #4401
Conversation
PF4 preview: https://patternfly-react-pr-4401.surge.sh |
12923d7
to
d9b04b0
Compare
packages/react-core/src/components/AlertGroup/__tests__/__snapshots__/AlertGroup.test.tsx.snap
Show resolved
Hide resolved
...src/components/ApplicationLauncher/__tests__/__snapshots__/ApplicationLauncher.test.tsx.snap
Show resolved
Hide resolved
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.
@redallen Please let me know the remediation steps for the issues I found
...src/components/ApplicationLauncher/__tests__/__snapshots__/ApplicationLauncher.test.tsx.snap
Outdated
Show resolved
Hide resolved
packages/react-core/src/components/Nav/__tests__/__snapshots__/Nav.test.tsx.snap
Show resolved
Hide resolved
packages/react-core/src/components/Pagination/__tests__/__snapshots__/Pagination.test.tsx.snap
Outdated
Show resolved
Hide resolved
packages/react-core/src/components/Pagination/__tests__/__snapshots__/Pagination.test.tsx.snap
Show resolved
Hide resolved
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.
LGTM. I don't see any data-ouia-component-type="PF4/undefined"
in the snaps
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.
Another round of conflicts to resolve, but LGTM!
7a88885
to
3329e1b
Compare
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.
Missing 2 ouia: true
s
packages/react-core/src/components/ApplicationLauncher/ApplicationLauncher.tsx
Show resolved
Hide resolved
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.
LGTM!
Your changes have been released in:
Thanks for your contribution! 🎉 |
Standardizes OUIA implementations and adds extra attrs for documentation linkage
Closes several pieces in #4400