Skip to content
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

feat: personal center validate #1932

Merged
merged 26 commits into from
Sep 13, 2024

Conversation

JoJohw
Copy link
Contributor

@JoJohw JoJohw commented Sep 11, 2024

修复个人中心验证码若干问题

  1. 验证码-错误提示没有展示对应的错误信息(bugfix)
  2. 验证码-兼容超长文本overflowTitle(feat)
  3. 验证码-必填项与错误信息文本重合(bugfix)
  4. 验证码-错误提示有缓存(bugfix)
  5. 验证码-邮箱发送不了验证码,一直弹必填项(bugfix)
  6. 验证码-手机号为空可以发送验证码(bugfix)
  7. 继承数据源切换自定义数据,在点击取消,重新打开select显示自定义数据,接口参数却显示继承(bugfix)

JoJohw added 26 commits August 27, 2024 15:20
# Reviewed, transaction id: 16907
postPersonalCenterUserEmailCaptcha接口
# Reviewed, transaction id: 16938
# Reviewed, transaction id: 16940
verifyIdentityInfoDialog组件下
# Reviewed, transaction id: 16970
# Reviewed, transaction id: 17108
# Reviewed, transaction id: 17112
# Reviewed, transaction id: 17181
并未更新为自定义和新邮箱的问题,前端提交请求is_inherited_email: true导致
# Reviewed, transaction id: 17228
# Reviewed, transaction id: 17558
# Reviewed, transaction id: 17559
fix: 修复空邮箱仍能弹出验证的问题
style: UI同步设计稿
# Reviewed, transaction id: 17832
# Reviewed, transaction id: 17846
1. 验证码-错误提示没有展示对应的错误信息(bugfix)
2. 验证码-兼容超长文本overflowTitle(feat)
3. 验证码-必填项与错误信息文本重合(bugfix)
4. 验证码-错误提示有缓存(bugfix)
5. 验证码-邮箱发送不了验证码,一直弹必填项(bugfix)
6. 验证码-手机号为空可以发送验证码(bugfix)
7. 继承数据源切换自定义数据,在点击取消,重新打开select显示自定义数据,接口参数却显示继承(bugfix)
# Reviewed, transaction id: 18094
# Reviewed, transaction id: 18100
@brookylin brookylin merged commit 025c47d into TencentBlueKing:main Sep 13, 2024
4 checks passed
@JoJohw JoJohw changed the title Feat/personal center validate feat: personal center validate Sep 27, 2024
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