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

A11y base tests #51893

Merged
merged 21 commits into from
Dec 12, 2019
Merged

A11y base tests #51893

merged 21 commits into from
Dec 12, 2019

Conversation

bhavyarm
Copy link
Contributor

@bhavyarm bhavyarm commented Nov 28, 2019

#51456

Under OSS

  • Visualize table
  • Create visualizions
  • Create dashboard
  • Dashboard listing table -
  • Add one sample data set
  • Create index pattern
  • Dev tools console

@bhavyarm bhavyarm mentioned this pull request Nov 28, 2019
77 tasks
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@bhavyarm bhavyarm added Project:Accessibility test-matrix Use this label to ensure PRs are tested with matrix jobs test_ui_functional release_note:fix release_note:skip Skip the PR/issue when compiling release notes and removed test_ui_functional release_note:fix labels Dec 12, 2019
@bhavyarm
Copy link
Contributor Author

@elasticmachine merge upstream

@bhavyarm
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💔 Build Failed

@lizozom
Copy link
Contributor

lizozom commented Dec 12, 2019

Seems to me this accidentally picked up some unrelated commits?

@bhavyarm
Copy link
Contributor Author

@lizozom yes. fixed now :) Thanks!

@bhavyarm
Copy link
Contributor Author

@elasticmachine merge upstream

@bhavyarm bhavyarm self-assigned this Dec 12, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

@spalger spalger removed the test-matrix Use this label to ensure PRs are tested with matrix jobs label Dec 12, 2019
@elastic elastic deleted a comment from elasticmachine Dec 12, 2019
@bhavyarm
Copy link
Contributor Author

@cjcenizal I have commented out the console test here because I want to merge this one now. We will have more PRs for a11y accessibility automation. Thanks! cc @myasonik

Copy link
Contributor

@myasonik myasonik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ship it!

@bhavyarm bhavyarm merged commit 4765ed0 into elastic:master Dec 12, 2019
bhavyarm added a commit to bhavyarm/kibana that referenced this pull request Dec 12, 2019
@bhavyarm bhavyarm added this to In progress in Test Automation via automation Dec 12, 2019
@bhavyarm bhavyarm moved this from In progress to Done in Test Automation Dec 12, 2019
bhavyarm added a commit that referenced this pull request Dec 13, 2019
timductive pushed a commit to timductive/kibana that referenced this pull request Dec 16, 2019
@kibanamachine
Copy link
Contributor

💔 Build Failed

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@elastic elastic deleted a comment from elasticmachine Mar 31, 2020
@spalger
Copy link
Contributor

spalger commented Mar 31, 2020

Hmm...

@myasonik
Copy link
Contributor

myasonik commented Apr 1, 2020

❓❓❓

@spalger
Copy link
Contributor

spalger commented Apr 1, 2020

Yeah, no clue why GHPRB decided it was time to test this PR again 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes v8.0.0
Projects
Development

Successfully merging this pull request may close these issues.

None yet

7 participants