-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Merge branch 'plugins' #160
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
Merge branch 'plugins' #160
Conversation
plugin objects definition, with two examples
Plugins object and a few plugins
Leaflet.Timedimension
Awesome! I really like your vision for this implementation. @themiurgo since you reviewed most of this PR how far are we from merging? |
@ocefpaf @themiurgo I'm a bit bolcked by this PR : if I want to go further on plugins, I don't know whether I shall commit them on the PS: sorry for that (most probably) silly question. |
Not at all! But since this is a big change lets wait for more eyes here. I am 👍 What do you say @themiurgo? |
I think either way is fine, as long as we keep master working. If you feel it's stable, merge it with master. If you want to have some more freedom to |
No description provided.