search

librenms-plugins / Weathermap

MIT License
64 stars 51 forks source link

WIP - A few more consequences of the Namespace #53

Open PipoCanaja opened 6 years ago

PipoCanaja commented 6 years ago

Back here. Seems that more changes are needed. @laf: do you think this is the right way to get the short class name (without namespace) or should it be done another way ?

At least this PR makes Weathermap completely operational now.

PipoCanaja commented 6 years ago

Will have to check https://github.com/librenms/librenms/pull/8917 as it may just solve and we revert to the original state.

PipoCanaja commented 6 years ago

Ok. Looks much better after the fix https://github.com/librenms/librenms/pull/8917 . Reverting the namespace.