nick.adam/certigo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Activity

Fix: Admin Permission Cancel Bug fixed #9

Merged
nick.adam merged 1 commits from fix/adminPermissionCancel-Bug into development 2025-11-22 02:31:15 +00:00
Conversation 0 Commits 1 Files Changed 1 +38 -5
nick.adam commented 2025-11-22 02:24:44 +00:00
Owner
Copy Link

Problem
Nach Klick auf die Admin-Checkbox öffnete sich die Warnung; bei "Abbrechen" blieb der Haken gesetzt.
Ursache
In handleAdminToggle wurde isAdmin sofort in formData gesetzt, bevor das Modal geöffnet wurde. Bei "Abbrechen" wurde nur das Modal geschlossen, der Wert blieb.
Lösung in frontend/src/pages/Users.jsx

  1. Neuer State
    pendingAdminChange speichert den vorherigen isAdmin-Wert
  2. handleAdminToggle
    Bei Aktivierung wird isAdmin nicht sofort gesetzt
    Vorheriger Wert wird in pendingAdminChange gespeichert
    Früher Return, Checkbox bleibt visuell unverändert
    Bei Deaktivierung wird sofort gesetzt
  3. Abbrechen-Button
    Setzt isAdmin auf pendingAdminChange
    Checkbox wird wieder ungecheckt
    pendingAdminChange auf null
  4. Bestätigen-Button
    Setzt isAdmin auf true
    Entfernt Gruppen, setzt enabled auf true
    pendingAdminChange auf null
  5. Cleanup
    pendingAdminChange wird an allen relevanten Stellen zurückgesetzt
Problem Nach Klick auf die Admin-Checkbox öffnete sich die Warnung; bei "Abbrechen" blieb der Haken gesetzt. Ursache In handleAdminToggle wurde isAdmin sofort in formData gesetzt, bevor das Modal geöffnet wurde. Bei "Abbrechen" wurde nur das Modal geschlossen, der Wert blieb. Lösung in frontend/src/pages/Users.jsx 1. Neuer State pendingAdminChange speichert den vorherigen isAdmin-Wert 2. handleAdminToggle Bei Aktivierung wird isAdmin nicht sofort gesetzt Vorheriger Wert wird in pendingAdminChange gespeichert Früher Return, Checkbox bleibt visuell unverändert Bei Deaktivierung wird sofort gesetzt 3. Abbrechen-Button Setzt isAdmin auf pendingAdminChange Checkbox wird wieder ungecheckt pendingAdminChange auf null 4. Bestätigen-Button Setzt isAdmin auf true Entfernt Gruppen, setzt enabled auf true pendingAdminChange auf null 5. Cleanup pendingAdminChange wird an allen relevanten Stellen zurückgesetzt
❤️ 1
nick.adam added 1 commit 2025-11-22 02:24:44 +00:00
Fix: Admin Permission Cancel Bug fixed 6623b07502
nick.adam requested review from mete.baser 2025-11-22 02:24:44 +00:00
nick.adam changed title from Fix: Admin Permission Cancel Bug fixed to WIP: Fix: Admin Permission Cancel Bug fixed 2025-11-22 02:24:54 +00:00
nick.adam changed title from WIP: Fix: Admin Permission Cancel Bug fixed to Fix: Admin Permission Cancel Bug fixed 2025-11-22 02:25:03 +00:00
nick.adam self-assigned this 2025-11-22 02:26:13 +00:00
nick.adam merged commit ec1e0da9d5 into development 2025-11-22 02:31:15 +00:00
nick.adam deleted branch fix/adminPermissionCancel-Bug 2025-11-22 02:31:15 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
mete.baser
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
mete.baser nick.adam
No Assignees nick.adam
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: nick.adam/certigo#9
No description provided.
Delete Branch "fix/adminPermissionCancel-Bug"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?