Skip to content

Enable System Monitor without reboots needed#10

Merged
MelchMwoan merged 2 commits into
mainfrom
patch-enable-sm-no-reboot
Mar 19, 2026
Merged

Enable System Monitor without reboots needed#10
MelchMwoan merged 2 commits into
mainfrom
patch-enable-sm-no-reboot

Conversation

@pieterjvt

Copy link
Copy Markdown
Member

No description provided.

@MelchMwoan MelchMwoan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • De error handling bij het inschakelen van de integratie werkte niet goed waardoor de addon crashte
  • Als de integratie disabled stond (niet de entiteiten. maar de integratie zelf) dan probeerde je die in te schakelen aan de hand van de flow-id wat niet werkt. Daar haalt die nu de juiste configentry op en schakelt die hem in aan de hand daarvan
  • Die connect/token error kwam doordat de API niet bereikbaar is op jouw laptop, maar daar hoort die uiteraard niet volledig van te crashen (dit is mijn fout overigens)
  • Voor de volgende keer, als je een nieuwe feature toevoegt aan de addon, gelieve dan ook bij de pr het versienummer op te hogen volgens SemVer (Major.Minor.Patch) en de CHANGELLOG.md bij te werken

@MelchMwoan MelchMwoan merged commit a71654a into main Mar 19, 2026
3 checks passed
@pieterjvt pieterjvt deleted the patch-enable-sm-no-reboot branch March 19, 2026 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants