Discuss: add config modal #20928
Discuss: add config modal #20928
Conversation
Reviewer's Guide by SourceryAdds a UI feature to generate GitHub discussions with system data and introduces a new API endpoint /api/system/live to expose the full live configuration. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
…sion for static and dynamic configurations
| if err := jsonDecoder(r).Decode(&payload); err != nil { | ||
| return loadpoint.DynamicConfig{}, nil, err | ||
| // Hilfsfunktion: Struct zu map[string]any | ||
| func structToMap(s any) map[string]any { |
|
Ohne im Detail geschaut zu haben: schön wäre, wenn sich der Anwender per Roundtrip bei GH anmelden könnte so dass wir per API direkt in seinem User die Discussion öffnen könnten. Ich hab allerdings nicht geprüft, ob das ohne redirect URL überhaupt klappen kann. |
| "startup": importSettings("startup"), | ||
| "plant": importSettings("plant"), | ||
| "telemetry": importSettings("telemetry"), | ||
| } |
There was a problem hiding this comment.
In vielen diese Entries können sensible Daten drin sein. Wir müssen hier auf jeden Fall sauber redact machen.
Hier ein paar stellen wo wir so etwas schon machen:
-
evcc dump
Line 68 in de40390
-
publishing von
mqtt,influx, ...
evcc/api/globalconfig/types.go
Line 88 in de40390
-
config api device Endpunkte (
vehicle, ...)
evcc/server/http_config_helper.go
Line 96 in de40390
| @@ -260,6 +260,7 @@ func (s *HTTPd) RegisterSystemHandler(site *core.Site, valueChan chan<- util.Par | |||
| "interval": {"POST", "/interval/{value:[0-9.]+}", settingsSetDurationHandler(keys.Interval)}, | |||
| "updatesponsortoken": {"POST", "/sponsortoken", updateSponsortokenHandler}, | |||
| "deletesponsortoken": {"DELETE", "/sponsortoken", deleteSponsorTokenHandler}, | |||
| "liveconfig": {"GET", "/live", LiveConfigHandler(site)}, | |||
There was a problem hiding this comment.
Ich bin mit dem Namen noch nicht glücklich. Ich wäre dafür das in den /system Namespace zu hängen. Also GET /system/config analog zu GET /system/log.
| @click="openDiscussModal" | ||
| > | ||
| {{ $t("help.discussButton") }} | ||
| </button> |
There was a problem hiding this comment.
Glaube den neuen Menüpunkt brauchen wir nicht. Es gibt ja bereits den "Bug melden" Menüpunkt. Der verlinkt momentan hart zu Github Issues. Der würde sich dafür ja super eigenen.
| return loadpoint.DynamicConfig{}, nil, err | ||
| } | ||
|
|
||
| return loadpoint.SplitConfig(payload) |
There was a problem hiding this comment.
Bin mir nicht sicher wo hier der Fehler ist, aber mit dieser Umstellung funktioniert die Loadpoint API nicht mehr. Siehe Integrationstests:
4 failed
[chromium] › tests/config-loadpoint.spec.js:66:3 › loadpoint › create, update and delete ───────
[chromium] › tests/config-loadpoint.spec.js:161:3 › loadpoint › priority ───────────────────────
[chromium] › tests/config-loadpoint.spec.js:215:3 › loadpoint › vehicle ────────────────────────
[chromium] › tests/config-loadpoint.spec.js:272:3 › loadpoint › keep mode ──────────────────────
| @@ -166,6 +298,64 @@ export default { | |||
| async restartConfirmed() { | |||
| await performRestart(); | |||
| }, | |||
| async openDiscussModal() { | |||
| const modal = Modal.getOrCreateInstance(document.getElementById("discussModal")); | |||
There was a problem hiding this comment.
Die Endpunkte hier erfordern Login. Hier müssen wir, analog Restart Modal, den Status prüfen.
evcc/assets/js/components/HelpModal.vue
Line 292 in de40390
| generatedData: "", | ||
| includeLogs: false, // Neue Variable für die Checkbox | ||
| logData: "", // Neue Variable für die Log-Daten | ||
| }; |
There was a problem hiding this comment.
Das DiscussModal sollte auf jeden Fall eine eigene Vue Komponenten werden. Da kommt ja einiges an State und Logik zusammen.
|
@diddip21 Danke für den PR. Ist ein großes Thema. Aber ist ein guter Start. Die Idee dem Nutzer die Daten strukturiert einmal in der evcc UI zu präsentieren bevor er sie zu Github rüberflankt ist gut. Gibt dem Nutzer mehr Kontrolle. Größtes Thema ist vmtl. die Redaktion/"Secrets schwärzen" sauber hinzubekommen. Hätte auch noch einige Anmerkungen zum Ablauf. Bspw. würde ich den |
#18874
#18772
Erster versuch eine Erste-Hilfe Discussion in Github zu eröffnen.
In der UI ist bis jetzt noch nicht die
api/config/livereferenziert da diese einen vorherigen login benötigt.Ansonsten ist es mein erster "armseliger" versuch eine Art von LiveConfig zu generieren.
Nehmt es auseinander 😎
Summary by Sourcery
Add a UI feature to assist users in creating GitHub support discussions and expose the live system configuration via a new API endpoint.
New Features:
/api/liveendpoint consolidating configuration from various sources (files, UI, database).Enhancements: