Skip to content

Conversation

@wangsijie
Copy link
Contributor

Summary

Check if CAPTCHA is verified before sending a verification code.

Testing

Integrationt ests.

Checklist

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

@github-actions
Copy link

github-actions bot commented Mar 27, 2025

COMPARE TO master

Total Size Diff 📈 +1.58 KB

Diff by File
Name Diff
packages/core/src/routes/experience/classes/experience-interaction.ts 📈 +8 Bytes
packages/core/src/routes/experience/verification-routes/verification-code.ts 📈 +59 Bytes
packages/integration-tests/src/tests/api/experience-api/captcha/index.test.ts 📈 +1.54 KB

Base automatically changed from wangsijie-log-10983-skip-for-social-and-sso-flow to master March 31, 2025 05:32
@github-actions github-actions bot added size/m and removed size/s labels Mar 31, 2025
@wangsijie wangsijie force-pushed the wangsijie-log-11063-guard-captcha-for-otp-sending branch from 4744614 to 23eca39 Compare April 1, 2025 01:15
@github-actions github-actions bot added size/s and removed size/m labels Apr 1, 2025
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.

overall LGTM

@wangsijie wangsijie merged commit d069541 into master Apr 1, 2025
37 checks passed
@wangsijie wangsijie deleted the wangsijie-log-11063-guard-captcha-for-otp-sending branch April 1, 2025 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants