-
Notifications
You must be signed in to change notification settings - Fork 936
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
Call for maintainers. I need your help #266
Comments
You have my sword on this path! 😄 |
I can help, also due to the fact I got to know the module a little bit better lately. |
Maintainers list
|
@tomchentw I'm thinking on helping you out too! And let me add that you when you say no project uses in production...we use it here: www.flykt.com !!! And I love it ^^ |
@tiagomegas thanks. Check out 6.0.0 which is released on npm beta tag now. We also have a new demo page. Feel free to try it: |
I'd be interested in helping to maintain this project. At Syncromatics, we're building out a new version of our site on top of this, and I'd like to be able to contribute improvements and help do the work of maintaining the codebase. How can I go about contributing? (I do already have a PR out for adding TrafficLayer) (CC @wuct @cristiandley @rewop) |
Happy to help also. I'm going to be using the project heavily for at least the next year so I'lll be around quite a bit. |
I work with @thzinc over at Syncromatics and as he mentioned we're using react-google-maps for the next version of our UI, so I'd like to contribute where I can as well. |
Happy to help guys. Going to be using this or something very close to it over the next couple of years. |
Hi guys. I would like to help you with it, going to use the project for the next year or so :) |
Ive got other commitments but I havent heard back from the author(no email response :( ), I'll be ramping up on this again come March if anyone wants to fork and continue |
Hey guys, there are a series of PR's that we can merge in and keep contributing to the project. @tomchentw any chance you can grant me access? |
There are 4 options for React Google maps. I created gist to compare them https://gist.github.com/stereobooster/13a26188d4ad5382bc6da9ffe76ce3e1. None of them actively supported :/. |
@stereobooster Yeah, because of that I ended up using react-leaftlet, which is well documented and fairly active. I may reconsider and go down the D3 route. |
Hello, I would like to help out! |
What's the status of this? |
of which @shinzui ? We are still looking for maintainers and pushing the project forward. @tomchentw has actually been fairly active helping out quite a bit. |
@oshalygin, sorry for the vague question. I'm picking a library to integrate google maps into my react app and saw the call for maintainers in the readme linking to this issue. The issue was opened 15 months ago so I was wondering if the project is going to have a maintainer steering it forward. |
@shinzui There are a number of maintainers that are keeping the project alive and we are constantly releasing new features with the help of the community 👍 Once we get some of the bigger pieces together with the project we'll cleanup the README a bit to remove ambiguity 💯 |
@oshalygin Are there any API docs?. All I could see are examples and with recompose. For example, i am trying to figure out what |
@sudheerDev the API docs for a component is documented in the Yeah, currently we don't have docs for |
@tomchentw Aah ok. Thanks. Just took time to figure out from examples, issues that few are components and few are higher order functions. Logs helped though Nice work with the repo! |
For anyone interested, feel free to jump in answering questions on StackOverflow first: |
Hi! What's the status on this issue? I'm currently working on a fork of the project but would be happy to be added as a maintainer/contributer instead. |
I'm also working on the fork, it will be ready this week, it does supports last API version, does not have componentWillMount - it uses getDerivedStateFromProps, does support all new features, and properly typed with prop-types (look ma, no PropTypes.any!), supports to load unlimited number of maps on the same page, has no lodash in dependencies and very lightweight and performant. I will publish it soon. |
@urikphytech if you want to join, I can give you access to the repo. |
@JustFly1984 That sounds amazing! I'd be glad to get access and help out with your open issues/tasks. I did some work on switching to the new context API, but right now it depends on react v16.5 because it's using Class.contextType. This is actually my work account, better invite @uriklar which is my personal account |
@urikphytech new context api doesn't help if you need multiple maps on the same page. I've fixed this issue. I will add you to collaborators. Please do your changes in separate branch, and make pull requests. Do you have skype or something? |
@urikphytech I've added you to collaborators, lets continue conversation here https://github.com/JustFly1984/react-google-maps-api/invitations It is private repo, cos I use my own key to test development. There is an example build on gatsby (so I could test SSR later on) |
I am working on a new React wrapper for Google Map API, based on the latest React features: hooks, context... Current status:
Highlights:
@urikphytech @JustFly1984 it is possible to have multiple maps on the same page with context. You only need to use multiple contexts. My Repo |
@lucifer1004 we have a stable API at |
Btw you can check out the hooks branch. It has a working poc of the useMapComponent hook We would love you to join us! We want to start adding test as well soon so it would be great to get you on board working on that |
For anyone interested, feel free to jump in answering questions on StackOverflow first:
https://stackoverflow.com/search?tab=newest&q=react-google-maps
General Reply Template
The text was updated successfully, but these errors were encountered: