-
Notifications
You must be signed in to change notification settings - Fork 154
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
fix update issuse by adding current ref #93
fix update issuse by adding current ref #93
Conversation
so, the I don't really understand what the @sinnrrr what do you think? |
@srghma looks good |
published at |
@srghma no, I don't have them too I believe you should have this page on top panel of the repo page because you have the "Collaborator" badge |
it shows here, that you were invited as a collaborator, you should probably check your email then |
no, I dont have any email) |
@srghma Sorry for the late reply. For some odd reasons I was not getting notified. You can add me as a repo maintainer. Thanks happy to contribute. |
#90
I found few bugs my above branch.
I fixed update logic, which was not executing properly.
I added
layer.addTo(map)
so that map works properly.I found this bug while working on my personal project and the problem I faced is mentioned here
found bug in my code
Note this bug was visible in CRA bootstrapped app, but in custom webpack config I was not able to detect this bug
@srghma @agusterodin Please review