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

feat(animatedaxis): add tickComponent support (#977) #1228

Merged
merged 1 commit into from
Jun 25, 2021
Merged

feat(animatedaxis): add tickComponent support (#977) #1228

merged 1 commit into from
Jun 25, 2021

Conversation

johnathanludwig
Copy link
Contributor

🚀 Enhancements

This adds support for the tickComponent prop on AnimatedAxis. It was already documented as being supported.

Copy link
Collaborator

@williaster williaster left a comment

Choose a reason for hiding this comment

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

Sorry for the delay in review here, this looks good to me 🏆

Thanks for the additino @johnathanludwig 🙌 !

@williaster
Copy link
Collaborator

williaster commented Jun 25, 2021

Ah looks like this is failing the CI from linting, can you run yarn lint:fix (and possibly yarn format to be safe) and then commit the changes?

@johnathanludwig
Copy link
Contributor Author

I rebased with master, ran lint:fix, and ran format. Those fixes resulted in changes to quite a few other files that I didn't originally touch, so I discarded those to keep this PR focused on the original changes.

@williaster
Copy link
Collaborator

awesome, thanks @johnathanludwig !

@williaster williaster merged commit c51fdf1 into airbnb:master Jun 25, 2021
@github-actions
Copy link

🎉 This PR is included in version v1.14.0 of the packages modified 🎉

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

Successfully merging this pull request may close these issues.

2 participants