Conversation
bissenbay
commented
Jul 27, 2019
bissenbay
commented
- Changed Google Maps to OpenStreetMap
- Retrieving GNO data from OpenStreetMap API
- Caching the data
|
@frederickjansen @shreyapandit It seems like the changes I made to Urban Refugee last year was not merged to the upstream. Please take a look! Thanks! |
|
Hey @bissenbay, development on this has been taken over by @nravic. I'm going to let him handle these changes. Thanks! |
@frederickjansen Nice. Thanks a lot! |
|
Hi @nravic! |
|
Hey Bissenbay,
Yeah! I'm focusing on refactoring the code, because we want a working db
for San Jose. Can't get the OSM guys in time to put all the info up, so I'm
reverting back to when the code used a database, and starting from there.
I'm gonna rewrite a lot of the map stuff, but leave the UI things in for
now.
…On Wed, Aug 21, 2019, 2:50 PM Bissenbay Dauletbayev < ***@***.***> wrote:
Hi @nravic <https://github.com/nravic>!
Any updates?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4?email_source=notifications&email_token=AADD536ZHLRRON2DIHLCE2LQFWE55A5CNFSM4IHKLNIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD42YEPI#issuecomment-523600445>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADD5335XGGJFLSTCLLDHI3QFWE55ANCNFSM4IHKLNIA>
.
|
|
@nravic Do you want to merge this, so my PR is closed? This was accepted last year. |
|
@nravic Could you please take a look and approve this PR to be merged. It was accepted by @weirdindiankid last year when we worked on this project in Summer time. I think you can then make changes on top of my changes. Thanks, 😃 ! |
|
Hi @frederickjansen , @nravic ! Can this PR get merged? |
1 similar comment
|
Hi @frederickjansen , @nravic ! Can this PR get merged? |
|
Hi @bissenbay I no longer work at BU so I have no control over pull requests. |
Sorry for bothering, if you know anyone by change in charge for this repo could you reach out to them, otherwise, it is OK, congrats on your new position. |
|
Hi @nravic, can my PR be merged? |
4 participants
