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.

Sign up for GitHub

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

Jump to bottom

建议邮件的“通知设置”添加一个“发送测试”的功能 #4714

Closed
GuGuGee opened this issue Oct 12, 2023 · 4 comments · Fixed by #5464
Closed

建议邮件的“通知设置”添加一个“发送测试”的功能 #4714

GuGuGee opened this issue Oct 12, 2023 · 4 comments · Fixed by #5464
Assignees
@LIlGG
Labels
area/core Issues or PRs related to the Halo Core kind/feature Categorizes issue or PR as related to a new feature.
Milestone
2.14.0

Comments

@GuGuGee
Copy link

GuGuGee commented Oct 12, 2023

Your current Halo version

2.10.0

Describe this feature

目前后台有提供“启用邮件通知器”的功能，配置完SMTP之后，没有一个类似之前1.6版本的“发送测试”功能，验证自己的SMTP配置是否正确，希望可以在之后的版本添加这个功能，非常感谢！

Additional information

No response
@JohnNiang
Copy link
Member

/kind feature
/area core

Thank you for reaching out here! We will try our best to recover the feature.

@f2c-ci-robot f2c-ci-robot bot added kind/feature Categorizes issue or PR as related to a new feature. area/core Issues or PRs related to the Halo Core labels Oct 12, 2023
@ruibaby ruibaby mentioned this issue Dec 8, 2023
Closed
@ruibaby
Copy link
Member

ruibaby commented Dec 8, 2023 •
edited

可能还需要为 FormKit 提供一个这样的组件。

@guqing guqing mentioned this issue Mar 4, 2024
Closed
@ruibaby ruibaby added this to the 2.14.x milestone Mar 4, 2024
@LIlGG
Copy link
Member

LIlGG commented Mar 6, 2024

可能还需要为 FormKit 提供一个这样的组件。

实际上我感觉只需要简单的扩展一下 Form 组件，例如将 示例 中的 @submit="login" 更改为一个内置的验证方法即可。

@LIlGG
Copy link
Member

LIlGG commented Mar 6, 2024

/assign

@LIlGG LIlGG mentioned this issue Mar 7, 2024
Merged
f2c-ci-robot bot pushed a commit that referenced this issue Mar 26, 2024
@LIlGG
feat: add verification function to the notification settings for the …
a281015 
…mailbox (#5464)

#### What type of PR is this?

/kind feature
/area ui
/area core
/milestone 2.14.x

#### What this PR does / why we need it:

为邮件的 `通知设置` 添加验证的功能。

同时为 formkit 增加了一个新的组件 (verificationForm)，用于支持验证，它的定义方式如下：
```
- $formkit: verificationForm
  action: "http://localhost:8090/verify/user"
  label: 用户验证
  children:
    - $formkit: text
      label: "用户名"
      name: username
      validation: required
    - $formkit: password
      label: "密码"
      name: password
      validation: required
```

verificationForm 支持 `action` 属性，当前端数据验证通过时，会将其下所包含的子节点数据发送至 action 所代表的接口上。
按上述示例，则验证数据会提交至 `http://localhost:8090/verify/user` 进行验证。验证的数据为 `{name: xxx, password: xxx}`

需要注意的是，verificationForm 只用于包装需要验证的数据，不会破坏原始数据的格式。因此上述数据在提交保存后仍旧为 `{name: xxx, password: xxx}` 而不会变成 `{verificationForm1: {name: xxx, password: xxx}}`

#### How to test it?

1. 测试邮箱中的 `通知设置` 新增的验证按钮是否可以正常验证邮箱。
2. 查看数据是否正常回显

#### Which issue(s) this PR fixes:

Fixes #4714 

#### Does this PR introduce a user-facing change?
```release-note
在邮件通知设置中增加了发送测试的功能。
```
@ruibaby ruibaby modified the milestones: 2.14.x, 2.14.0 Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LIlGG LIlGG

Labels
area/core Issues or PRs related to the Halo Core kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Milestone
2.14.0
Development

Successfully merging a pull request may close this issue.

feat: add verification function to the notification settings for the mailbox LIlGG/halo
4 participants
@JohnNiang @ruibaby @LIlGG @GuGuGee