Skip to content

GitHub Issue 848: Add the ability to reset TOTP settings in the apps#1981

Merged
XingY merged 6 commits intodevelopfrom
fb_totpReset
Apr 17, 2026
Merged

GitHub Issue 848: Add the ability to reset TOTP settings in the apps#1981
XingY merged 6 commits intodevelopfrom
fb_totpReset

Conversation

@XingY
Copy link
Copy Markdown
Contributor

@XingY XingY commented Apr 14, 2026

Rationale

Issue link
This PR add the ability for app admins to reset a user's TOTP (Time-based One-Time Password) settings from the user management panel.

Related Pull Requests

Changes

  • refactored UserDetailsPanel from class to FC
  • new UserResetTotpSettingsConfirmModal
  • api actions for get/reset TOTP settings

);
};

UserDetailsPanel.displayName = 'UserDetailsPanel';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just curious if you did the conversion from a class to FC here or if Claude code did? It would be nice to get more of these converted so if CC is ready to take on that task, we can add that to our "do this when you touch a component" list.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This one was actually done by Junie (free no subscription). I didn't try CC because I had good experience using Junie to do these type of conversion, but I expect CC to do a good job on this one too.

@XingY XingY merged commit 361ac6b into develop Apr 17, 2026
3 checks passed
@XingY XingY deleted the fb_totpReset branch April 17, 2026 16:54
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