Skip to content

[stable33] fix(settings): confirm app-token revoke and preserve wipe state#60768

Open
pringelmann wants to merge 7 commits into
stable33from
backport/60644/stable33
Open

[stable33] fix(settings): confirm app-token revoke and preserve wipe state#60768
pringelmann wants to merge 7 commits into
stable33from
backport/60644/stable33

Conversation

@pringelmann
Copy link
Copy Markdown
Contributor

@pringelmann pringelmann commented May 27, 2026

Backport of #60644

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@pringelmann pringelmann marked this pull request as ready for review May 27, 2026 13:22
@pringelmann pringelmann requested review from a team as code owners May 27, 2026 13:22
@pringelmann pringelmann requested review from artonge, nfebe, salmart-dev, sorbaugh and susnux and removed request for a team May 27, 2026 13:22
@pringelmann pringelmann self-assigned this May 27, 2026
@pringelmann pringelmann added enhancement 3. to review Waiting for reviews feature: settings team: IDP Code maintained by 🪪 IDP team labels May 27, 2026
@pringelmann pringelmann added this to the Nextcloud 33.0.4 milestone May 27, 2026
Adds an NcDialog confirmation to the Revoke action; deletion was
previously instant on click. When the token is marked for remote
wipe, the dialog surfaces a warning and the destructive button
switches to "Cancel wipe and revoke" so cancelling an in-flight
wipe is an explicit opt-in.

Also migrates the existing Wipe confirm from the legacy
window.OC.dialogs.confirm helper to NcDialog, matching the new
delete dialog. The auth token store actions are now pure API
callers; the UI does the gating.

Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
@pringelmann pringelmann force-pushed the backport/60644/stable33 branch from a77ab05 to 55a11b9 Compare May 27, 2026 13:43
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
@pringelmann pringelmann force-pushed the backport/60644/stable33 branch 2 times, most recently from 7b87c30 to 40989a0 Compare May 27, 2026 19:28
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement feature: settings team: IDP Code maintained by 🪪 IDP team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants