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

[RFC] Library Roadmap #552

Open
10 of 15 tasks
bell-steven opened this issue May 13, 2020 · 7 comments
Open
10 of 15 tasks

[RFC] Library Roadmap #552

bell-steven opened this issue May 13, 2020 · 7 comments

Comments

@bell-steven
Copy link
Collaborator

bell-steven commented May 13, 2020

Short term goals:

  • Add a table with a list of all available props.
  • Review all of the open PRs
  • Fix critical bugs in the library
  • Using Current location is broken in RN v0.60.0+ (Geolocation umbrella issue #541)
  • Triage all of the open issues
  • Enable web support (by using a proxy server)
  • add typescript definitions

Long terms goals:

  • Provide a clear write up about the difference Google APIs this library utilizes.
  • Bring the codebase inline with some of newer react/react-native features and patterns.
  • Update typescript definitions, propTypes and Readme with all of the available props.
  • Update documentation with clear examples
  • Update Examples in the example folder
  • modernize the codebase so that the library is easier to work with and contribute to.
  • Cut down on the number of props (there are currently 41 props available)
  • Explore splitting up the component based on the type of Google Maps API you want to use.

If you are willing to contribute by taking a task and creating PRs, that would be most appreciated (and will really speed things up). Sound off below

@ynigoreyes
Copy link
Contributor

Hi! Which props are you thinking of getting rid of? Where can I find more information about getting started on that one? @bell-steven

@bell-steven
Copy link
Collaborator Author

bell-steven commented Jun 19, 2020

@ynigoreyes I have a google sheet you can view here. It's still a work in progress. Would love your input. I can invite you to edit. Work will be done in the v2 branch, I just have to push it from my computer.

@strdr4605
Copy link
Contributor

strdr4605 commented Jul 7, 2020

Look into splitting up the component based on the type of Google Maps API you want to use.

I think this library serves only one propose and I don't see any reasonable splitting! Can you elaborate?!

@bell-steven
Copy link
Collaborator Author

@strdr4605 the library in its current state can do a few things.

  1. autocomplete an address/place search
  2. find nearby places
  3. get address from current location
  4. get place details from address/place search

Each of these functions uses a different api, and has unique query parameters. I wrote a pretty detailed breakdown of all of this here.

The idea would be to 'split' some of the functionality up to make it easier to use the library.
As an example

import { GooglePlacesAutocomplete } from "react-native-google-places-autocomplete"

might turn into something like

import { GetNearbyPlaces, GoogleAutoComplete, GetAddressFromLocation } from "react-native-google-places-autocomplete"

@bell-steven
Copy link
Collaborator Author

I created a v2 branch. Please create all breaking/major changes against that branch.

@QuintonC

This comment has been minimized.

@bell-steven

This comment has been minimized.

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

No branches or pull requests

4 participants