Skip to content

fix(core,console): fix captcha deletion behavior #7242

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

wangsijie
Copy link
Contributor

Summary

Fix captcha provider deletion behavior:

  1. Disable in SIE when deleted, enable when added.
  2. Reload front-end page when delete, because we are unable to recover the data.

Testing

Local tested.

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

Copy link

github-actions bot commented Apr 7, 2025

COMPARE TO master

Total Size Diff 📈 +304 Bytes

Diff by File
Name Diff
packages/console/src/pages/CaptchaDetails/index.tsx 📈 +223 Bytes
packages/console/src/pages/Security/Guide/index.tsx 📈 +131 Bytes
packages/console/src/pages/Security/use-data-fetch.ts 📈 +16 Bytes
packages/core/src/routes/captcha-provider/index.ts 📈 +380 Bytes

Copy link
Contributor

@simeng-li simeng-li left a comment

Choose a reason for hiding this comment

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

core changes LGTM

@simeng-li simeng-li merged commit 0b85868 into master Apr 9, 2025
35 checks passed
@simeng-li simeng-li deleted the wangsijie-log-11182-captcha-is-still-required-after-removing-captcha-service branch April 9, 2025 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants