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

Some updates that makes this library to work with the newest React versions + some bug fixes #92

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MohaElder
Copy link

  1. It seems that in newer versions of React(e.g: 18.x.x), you need to use PropsWithChildren to wrap your function or else there will be Property 'children' does not exist on type 'IntrinsicAttributes & Props' error.

reference: Answer from StackOverflow

  1. I noticed that there are missing prop type definitions in Marker (title & subtitle) and Map (tintColor). I have added them.

@MohaElder
Copy link
Author

Hi! I saw your comment on this closed PR that explained why you don't want to upgrade library's React related specifications to ensure its compatibility with older React versions. But I was wondering if it is possible to have a seperate version of this library that supports newer react versions since the current library crashes on newer react versions. If possible, I could help with that! Thanks for the time reading this~

@chrisdrackett
Copy link
Owner

I think this library could probably be updates to both not crash and work on older react versions :D

I plan on circling back to this library soon as I need it for an upcoming project. In the meantime I'm happy to accept PRs.

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.

2 participants