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

chore: make react-dom optional peerDep #5507

Open
wants to merge 2 commits into
base: canary-patch
Choose a base branch
from

Conversation

proohit
Copy link

@proohit proohit commented Oct 25, 2022

Summary

fixes #5503

Changelog

[Changes] - make react-dom an optional peerDependency

@auto-assign auto-assign bot requested a review from surajahmed October 25, 2022 07:07
@vercel
Copy link

vercel bot commented Oct 25, 2022

@proohit is attempting to deploy a commit to the Geekyants Team Team on Vercel.

A member of the Team first needs to authorize it.

@Viraj-10 Viraj-10 changed the base branch from master to canary-patch October 25, 2022 08:12
@Viraj-10
Copy link
Collaborator

Thanks for your contribution @proohit 😄.

@proohit
Copy link
Author

proohit commented Oct 25, 2022

Yep, should also mention that this fixes #5391

@alessandrocapra
Copy link

Is this PR still to be merged? It could fix many issues users are reporting

@proohit
Copy link
Author

proohit commented Nov 28, 2022

@alessandrocapra it is yet to be merged, but requires authorization by the NativeBase team to do so.

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

Successfully merging this pull request may close these issues.

Make react-dom optional peerDependency
4 participants