-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
fix: types for Cypress.Commands.add (#20376) #20377
Conversation
Thanks for taking the time to open a PR!
|
97456df
to
87091cd
Compare
87091cd
to
f6e6edd
Compare
@atarek12 Thanks for opening this PR. Our team will pick this up as soon as possible. |
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.
Looks good to me; Just rerunning some of our flaky tests to get everything green before merging.
@atarek12 This PR caused a type failure when merged in and was reverted, can you please re-open this PR against the latest |
…e-config * 10.0-release: chore: Move component-index generation to scaffold-config package (#21090) fix: label text should be clickable to toggle snapshot highlight (#21122) feat: add next preset to webpack-dev-server-fresh (#21069) chore: add dev-servers as deps to server to be included in the binary (#21142) fix: do not highlight preExtension if selected option is renameFolder (#21121) fix(unify): Remove 'Reconfigure' dropdown from Testing Type chooser (#21063) feat(unify): launchpad header breadcrumbs and reusable tooltip component (#20648) test: add windows-test-kitchensink job (#20847) fix: aut centering and height calculation (#21019) chore: fix tests that fail on windows (#21055) fix: running a new test after already having run tests (#21087) fix: ct project setup framework keys for next and nuxt (#21116) fix: remove MountReturn from scaffolded ct support file (#21119) fix: do not scaffold fixtures if folder exist (#21078) fix: revert "fix: types for Cypress.Commands.add (#20376)" (#21104) chore: Update Chrome (stable) to 100.0.4896.127 and Chrome (beta) to 101.0.4951.34 (#21083) fix: types for Cypress.Commands.add (#20376) (#20377)
User facing changelog
Fix
Cypress.Command.add()
TypeScript types. Fixes #20376element
while usingprevSubject: true
optionselector
to the subject while usingprevSubject: true | "element"
optionAdditional details
This change fixes typescript errors while using the
Cypress.Command.add()
with { prevSubject: true } or { prevSubject: "element" }How has the user experience changed?
No typescript error while trying to access the selector.
PR Tasks
cypress-documentation
?type definitions
?cypress.schema.json
?