Skip to content

Conversation

@Maschga
Copy link
Collaborator

@Maschga Maschga commented Oct 31, 2025

Fix #24557

grafik

@Maschga Maschga added the ux User experience/ interface label Oct 31, 2025
@Maschga Maschga changed the title fix: skip confirm backup dialog when auth disabled fix: skip backup confirmation in disabled-auth mode Oct 31, 2025
@Maschga Maschga marked this pull request as ready for review October 31, 2025 17:00
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey there - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@andig andig requested a review from naltatis October 31, 2025 18:25
@andig andig changed the title fix: skip backup confirmation in disabled-auth mode Skip backup confirmation in disabled-auth mode Oct 31, 2025
@naltatis
Copy link
Member

naltatis commented Nov 2, 2025

Welcher Request ist das und von wo aus greifst du auf evcc zu?

Lokal via vite-dev (port 7071). Das scheint auch das Problem zu sein. Wenn ich einen Build mache und direkt über die Go-Anwendung (7070) arbeite funktioniert der /api/system/backup Request korrekt. Weißt du woran das liegt, dass der API-Call durch den Vite Proxy nicht funktioniert? Ich meine das ging mal.

@naltatis
Copy link
Member

naltatis commented Nov 2, 2025

@Maschga Lass uns hier nicht den Flow für den Disable Auth Fall ändern. Das macht den Test auch weniger aussagekräftig. Hat auch den Effekt, dass der "Download..." Button, dessen "..." ja symbolisiert, dass noch ein Dialog kommt, sonst geändert werden müsste.

Das Issue hier ist ja reine Kommunikation. Ich würd vorschlagen einfach den Inhalt des Confirm Dialogs zu ändern. Im einfachsten Fall den markierten Bereich einfach entfernen.

Bildschirmfoto 2025-11-02 um 09 00 18

@naltatis
Copy link
Member

naltatis commented Nov 2, 2025

Strange. Ich habe gerade kommentiert (hier), aber der Kommentar steht jetzt ganz oben 👀

@naltatis
Copy link
Member

naltatis commented Nov 2, 2025

Anders, verwandtes Thema. Bei mir funktioniert der Backup download aktuell nicht. Ich sehe im Netzwerk einen Request, der terminiert aber nie. Ist hier was geändert worden?

\cc @andig

@Maschga
Copy link
Collaborator Author

Maschga commented Nov 2, 2025

Strange. Ich habe gerade kommentiert (hier), aber der Kommentar steht jetzt ganz oben 👀

Bei mir auch. Die gesamte History ist verdreht.

@Maschga
Copy link
Collaborator Author

Maschga commented Nov 2, 2025

Weißt du woran das liegt, dass der API-Call durch den Vite Proxy nicht funktioniert?

Nicht genau. Aber ich gehe davon aus, dass das das gleiche Problem wie bei der App ist.

@Maschga
Copy link
Collaborator Author

Maschga commented Nov 2, 2025

Hat auch den Effekt, dass der "Download..." Button, dessen "..." ja symbolisiert, dass noch ein Dialog kommt, sonst geändert werden müsste.

Gutes Argument.

Ich sehe im Netzwerk einen Request, der terminiert aber nie. Ist hier was geändert worden?

Welcher Request ist das und von wo aus greifst du auf evcc zu?

@andig
Copy link
Member

andig commented Nov 2, 2025

Anders, verwandtes Thema. Bei mir funktioniert der Backup download aktuell nicht. Ich sehe im Netzwerk einen Request, der terminiert aber nie. Ist hier was geändert worden?

Nicht dass ich wüsste. Ich kenne das Problem aber auch- tw. nur halbe DB erhalten mit nur einer Tabelle. War aber nicht reproduzierbar.

@Maschga Maschga changed the title Skip backup confirmation in disabled-auth mode Backup: update translations to not mention password input Nov 5, 2025
@Maschga Maschga requested a review from naltatis November 7, 2025 14:35
"backup": {
"action": "Stáhnout zálohu...",
"confirmationButton": "Stáhnout zálohu",
"confirmationText": "Prosím, zadejte své heslo pro stažení databázového souboru.",
Copy link
Member

Choose a reason for hiding this comment

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

The delete is not necessary. Weblate should mark this as "to be adjusted" when source translation changes. Deleting it might introduce more conflicts.

@naltatis naltatis merged commit d7b4647 into evcc-io:master Nov 12, 2025
8 checks passed
@Maschga Maschga deleted the fix/disable-auth-backup branch November 12, 2025 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ux User experience/ interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Backup UI: remove password communication when --disable-auth

3 participants