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

findBy* for #345 #346

Merged
merged 3 commits into from
Apr 5, 2019
Merged

findBy* for #345 #346

merged 3 commits into from
Apr 5, 2019

Conversation

joual
Copy link
Contributor

@joual joual commented Apr 5, 2019

What:

Why:

How:

Checklist:

  • Documentation added to the
    docs site
  • Tests
  • Typescript definitions updated
  • Ready to be merged

Closes #345
Closes #347

kentcdodds
kentcdodds previously approved these changes Apr 5, 2019
alexkrolick
alexkrolick previously approved these changes Apr 5, 2019
@kentcdodds
Copy link
Member

This actually isn't enough to fix it for 16.9.0 unfortunately. I'll go ahead and pull this PR and work on it a bit more. Thanks!

@joual
Copy link
Contributor Author

joual commented Apr 5, 2019

Ya I figured, and you might want to put the test somewhere that makes more sense or massage it into something that might detect all the cases you need :)

@joual
Copy link
Contributor Author

joual commented Apr 5, 2019

Thank you for all your work!

@joual joual closed this Apr 5, 2019
@kentcdodds
Copy link
Member

Oh, let's keep this open. I'll just update your PR :)

@kentcdodds kentcdodds reopened this Apr 5, 2019
@kentcdodds kentcdodds dismissed stale reviews from alexkrolick and themself via c660280 April 5, 2019 19:40
@kentcdodds
Copy link
Member

@allcontributors[bot], please add @joual for bug, test, and code

@allcontributors
Copy link
Contributor

@kentcdodds

I've put up a pull request to add @joual! 🎉

@kentcdodds kentcdodds merged commit 021ef9d into testing-library:master Apr 5, 2019
@joual joual deleted the failed-find-by branch April 5, 2019 19:46
@kentcdodds
Copy link
Member

🎉 This PR is included in version 6.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

lucbpz pushed a commit to lucbpz/react-testing-library that referenced this pull request Jul 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants