-
Notifications
You must be signed in to change notification settings - Fork 339
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
Merged a lot of PRs #80
base: 2.0/develop
Are you sure you want to change the base?
Conversation
Following my issue #59.
Cloudmade tile not working. Replaced by OpenStreetMap
Update README.md: Using Square Markers
@bhaskarvk can you direct that to the new repo? |
Will do |
Actually I can't, because your repo is not a proper fork of original repo. So GH is not allowing me to create a PR on your repo. Could you make your repo a fork of the original repo, that way I can submit a PR on your repo. |
@bhaskarvk I asked GitHub to detach the fork, because without that, I would not be able to create issues. If it's not too much work, I can give you permission on the repo, and you can merge (if everything is tested and ok), directly in the repository. That would make you a contributor as well ;) You basically would just add a new remote and push to it. |
Could I make an alternate suggestion ? Even if your repo is detached from this repo, you can still add this repo as an upstream repo to your local checkout.
Then open .git/config and add So it should look something like
Then do |
@bhaskarvk @george-silva What is the status on this? Are you going to put these changes in the new repo? |
@theashyster the new repo is already setup with releases to npm and bower. I haven't merged the PR from @bhaskarvk . @bhaskarvk can you PR to that repo instead? |
@george-silva Maybe you can do the suggestion from this comment? #80 (comment) |
The PR is open at: sigma-geosistemas/Leaflet.awesome-markers#8 Please, help me with testing 😄 Thanks everyone. |
@bhaskarvk just fyi, you can 'turn on' issues in a fork via: Settings > Features
|
@jgravois That's for @george-silva to do on his repo. My fork is not the official successor, his is. |
I've merged PRs
#46
#48
#50
#51
#60
#75
#76
#77
#78
and also fixed the examples.