-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
a11y for xpack home #62342
a11y for xpack home #62342
Conversation
will change the status to ready once I get a pass through CI. |
src/plugins/home/public/application/components/feature_directory.js
Outdated
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!
Will you also comment on the issues you opened when this merges leaving a note for who ever takes the issue to also un-skip the tests?
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.
Great Tests- covers a lot of ground.
LGTM .
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.
Code LGTM, thx a lot for those tests!
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
a11y test xpack home
Meta:
#51456
The pr goes into xpack home, checks out all the plugins in all, visualize & explore, admin category, opens a kibana app, navigates back to home, opens an add data tutorial, checks out the instructions on self managed and then opens instructions for tutorial on cloud
Skipped tests are because of these two issues:
#62239
#38980
There was no data-test-subj infrastructure. Will talk to @LeeDr to see what we want to do about missing functional tests.