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

Cypress: Force readonly inputs click #4016

Merged
merged 1 commit into from
Jul 30, 2019
Merged

Conversation

gabrieldutra
Copy link
Member

What type of PR is this? (check all applicable)

  • Other

Description

After cypress update to v3.4.1 readonly inputs are not reachable by default, so I added { force: true } where needed to make existing CI tests pass again.

Related Tickets & Documents

--

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

--

@gabrieldutra gabrieldutra self-assigned this Jul 29, 2019
@ranbena ranbena merged commit eae1fb7 into master Jul 30, 2019
@ranbena ranbena deleted the cypress-readonly-inputs branch July 30, 2019 08:19
@gabrieldutra
Copy link
Member Author

This error felt a bit weird to me, so I reached Cypress and it is actually supposed to only affect cy.type. Issue: cypress-io/cypress#4874

We can revert this once they solve it :)

@ranbena
Copy link
Contributor

ranbena commented Jul 30, 2019

I knew it!
I read that it was for type and not click so i opened their code and then said to myself - forget it, you don't have to know everything!

gabrieldutra added a commit that referenced this pull request Nov 13, 2019
gabrieldutra added a commit that referenced this pull request Nov 13, 2019
harveyrendell pushed a commit to pushpay/redash that referenced this pull request Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants