Is this still maintained?

Question

Is this still maintained?

zoolle opened this issue 3 years ago · 12 comments

I can see that there is MapBox Navigation SDK 2.0 but I haven't seen an update of this project for couple of months is someone still working on it?

Answer 1 · 2021-12-17T15:24:02.000Z

I think it is a great question and is important to know for some projects that are in progress

Answer 2 · 2021-12-20T18:03:50.000Z

This project is still used in production, just not in active development. Pull requests are welcome. We are trying to dedicate time to the active development of this library and will hopefully have some progress in the coming months.

Answer 3 · 2021-12-23T08:12:42.000Z

Hi @joe307bad who do i talk to if i wish to contribute?

Answer 4 · 2021-12-23T13:41:22.000Z

Hey @haizenburg, thanks for reaching out. I am the point of contact for contributing to this repo. Please feel free to open a PR and I'd be happy to review and merge ☺️

Answer 5 · 2022-01-14T13:24:38.000Z

Hey @joe307bad, I will try to add a couple of props that might be helpful for everyone else. My question is how do you guys test your implementation before making the PR?

Answer 6 · 2022-01-14T21:49:23.000Z

Hey @JorgeQuevedoC, thats a good question. I would say that any PR would probably benefit from also adding an example app to this project? That seems like the best way to test any changes. If someone were to put up a pull request I would be happy to make some time to also a minimal example project.

Answer 7 · 2022-01-18T15:20:16.000Z

Hi @joe307bad I have done some changes but I don't know how to test them on my project before I create the PR. Any suggestion?

And beside my changes I would like to help to migrate the library to use "Mapbox Navigation 2.0.0" since there are a lot of bugs that have been fixed

Answer 8 · 2022-01-18T16:11:02.000Z

Hey @JorgeQuevedoC, thanks for taking the time to work on this, I really appreciate it.

I would say the best way to test is to add an /example directory to the root of this repo and use npx react-native init to add an example application. Once the example app is created, we can create a basic implementation of react-native-mapbox-navigation within this repo. I have seen other packages do this and I think its probably the best practice, what do you think think?

Also, I wouldn't worry about testing your changes before posting the PR. Feel free to post the PR without an example app and I would be happy to work with you to figure out the best way to test.

Answer 9 · 2022-01-18T19:00:35.000Z

@joe307bad would you mind having a call to ask you a couple of thing about it?

Answer 10 · 2022-01-18T19:48:10.000Z

Hey @JorgeQuevedoC, I would be more than happy to have a call but lets exhaust some options before we try to coordinate a phone call. Could you post your PR and allow me to take a look? Do you have a specific error you may be running into?

Answer 11 · 2022-01-18T20:30:32.000Z

@joe307bad There are no errors because I don't know how to run the changes locally to see if it works

Answer 12 · 2022-01-18T22:19:21.000Z

Thanks @JorgeQuevedoC, feel free to post a PR (even if the code is a work in progress) and we can communicate about any new code within that PR. I would be happy to coordinate a call when I can look at some code and formulate a strategy for testing the changes you would like to make 😄