Ravi Kumar Kempapura Srinivasa
Ravi Kumar Kempapura Srinivasa
@KornesWerkman thank you for reporting this. I have already fixed this dependency issue here. You could try testing the module with the updated code.
@verboEse I have updated the code with a fix for this. Thank you for bringing it to my attention. You could try using the updated code.
@RincewindsHat : No special reason. I thought that these changes should be in the same commit. I have made the following changes in the class `Icinga\Module\Grafana\ProvidedHook\Icingadb\IcingaDbGrapher` 1. `$object->checkcommand` to `$object->checkcommand_name`...
@verboEse No, your config is correct. Instead of using class `Icinga\Module\Grafana\ProvidedHook\Icingadb\IcingadbSupport`, I was using `Icinga\Module\Graphite\ProvidedHook\Icingadb\IcingadbSupport` in `IcingadbGrafanaController`. Thank you for reporting this. I have fixed this issue now.
@verboEse I have fixed the issue. Could you try testing it again? Let me know if you face other issues. Thank you.
@sklaes You can find the bug description here: #135.
> Can you also please remove the right margin in tree view. Thanks! This change will be performed in a new PR as this is not related to the issue...
> The tiles are fine now. Though, we also decided to only show a single icon in the tree view. That's still missing. This is implemented in #352.
> Breadcrumb still has the issue. Did you reload the page after switching to branch? As there are some css changes present.
> * Please `limit` the `Display Name` with the help of validator. As discussed offline we will not be limiting `Display Name` length.