Skip to content

fix(Slider): do not pass infinite to div #208

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

Merged
merged 1 commit into from
Nov 14, 2019

Conversation

CarsonF
Copy link
Contributor

@CarsonF CarsonF commented Nov 13, 2019

What:
Not pass infinite prop to div

Why:
It's not a valid prop

How:
By destructing it out

Checklist:

  • Documentation added/updated (N/A)
  • Typescript definitions updated (N/A)
  • Tests added and passing
  • Ready to be merged

Introduced with #204

Copy link

@nicksrandall nicksrandall left a comment

Choose a reason for hiding this comment

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

LGTM

@bcarroll22
Copy link
Contributor

Thanks! @allcontributors, please add @CarsonF for code

@allcontributors
Copy link
Contributor

@bcarroll22

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

@bcarroll22
Copy link
Contributor

Also, @allcontributors add @nicksrandall for review

@allcontributors
Copy link
Contributor

@bcarroll22

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

@bcarroll22 bcarroll22 merged commit 4ec674d into express-labs:master Nov 14, 2019
@CarsonF CarsonF deleted the bugfix/slider-infinite branch November 14, 2019 15:02
@CarsonF
Copy link
Contributor Author

CarsonF commented Nov 16, 2019

@bcarroll22 Thanks man. Any chance I could get a patch release?

@bcarroll22
Copy link
Contributor

@tim-steele or @mrbinky3000 can we get a patch release for @CarsonF?

@bcarroll22
Copy link
Contributor

🎉 This PR is included in version 1.24.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants