-
Notifications
You must be signed in to change notification settings - Fork 1.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
[Banner] [Card] [Modal] convert banner legacy context API to use create context #786
Conversation
Yay! Our first item under the new "Code quality" changelog category :) |
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.
Thanks for the 🎩 links 🎉 LGTM, left a small nitpick and it might need a commit squash 💯
Sidenote soon we won't need to pass generics too all our HoC
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.
👍 🚢 following Andrews suggestion.
c7c917a
to
b4acf7d
Compare
b4acf7d
to
26d4af3
Compare
revert [Banner] [Card] [Modal] convert banner legacy context API to use create context #786
…API to use create context #786"
Addresses #785.
🎩