last release build v0.8.8 breaks the app

[mravinale]

Version:
https://github.com/tombatossals/angular-leaflet-directive/releases/tag/v0.8.8

JsFidle:
using the V0.8.8 http://jsfiddle.net/mravinale/rjnf0g78/

Error detail:

Error: [$injector:modulerr] Failed to instantiate module billboarderApp due to:
    Error: [$injector:modulerr] Failed to instantiate module leaflet-directive due to:
    Error: [$injector:modulerr] Failed to instantiate module nemLogging due to:
    Error: [$injector:nomod] Module 'nemLogging' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.

[mravinale]

fixed for now going back to the last version, be careful

[nmccready]

@mravinale look at the dependency list in bower the logger is now a dependency. If you use https://github.com/ck86/main-bower-files to load your deps you'll never have this problem

[nmccready]

also the error is flat out in there Module 'nemLogging' is not available!

[nmccready]

#829

[max-degterev]

some people use npm over bower for simplicity's sake. can we fix this somehow?

[max-degterev]

p.s. good god, angular-simple-logger depends on other modules as well, I wonder how long would I have to spend to load whole deps tree for this bugger

[nmccready]

People make mistakes and I am fixing good god, you don't have to be an db. You could also make a PR on the logger as well.

[elesdoar]

@suprMax Sorry, you could fork the project and contribute, or only use a previous version, don't make things more complicated. It's open source, all we contribute, not demand.

[max-degterev]

I mean no disrespect and appreciate the work you've done. Sorry if you've taken this personally.

No offense guys, just would be cool to have it working out of the box if it's published to npm, that's all.

[nmccready]

Thanks for the reply, and please remember tone is everything. I myself need to watch this from time to time. But when it is negative it is very very demotivating to continue working on these projects (when it is free).

There is actually a conversation going on now on leaflet 1.0 on becoming npm centric because leaflet itself wants out bower. Currently jspm, npm and keeping bower are on the table. #851

[nmccready]

Feel free to chime in and help out.