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

需要一个消息通知机制 #4045

Closed
ruibaby opened this issue Jun 7, 2023 · 5 comments · Fixed by #4527
Closed

需要一个消息通知机制 #4045

ruibaby opened this issue Jun 7, 2023 · 5 comments · Fixed by #4527
Assignees
@guqing
Labels
area/console Issues or PRs related to the Halo Console area/core Issues or PRs related to the Halo Core kind/feature Categorizes issue or PR as related to a new feature.
Milestone
2.10.0

Comments

@ruibaby
Copy link
Member

ruibaby commented Jun 7, 2023

你当前使用的版本

2.7.0

描述一下此特性

目前 Halo 2.x 还没有任何的消息通知功能，一些需要通知的场景：

  1. 注册验证
  2. 评论通知
  3. 重置密码

所以期望能够添加此功能。

附加信息

No response
@ruibaby
Copy link
Member Author

ruibaby commented Jun 7, 2023

/kind feature
/area core
/area console

@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 area/console Issues or PRs related to the Halo Console labels Jun 7, 2023
@LIlGG LIlGG added this to the 2.7.x milestone Jun 7, 2023
@ruibaby ruibaby modified the milestones: 2.7.x, Backlog Jun 22, 2023
@ql533cocou
Copy link

那个邮件插件不能发送通知吗？我不知道，我还没有用过那个邮件插件。

@ruibaby
Copy link
Member Author

ruibaby commented Jun 28, 2023 •
edited
Loading

那个邮件插件不能发送通知吗？我不知道，我还没有用过那个邮件插件。

通知还是交给 Core 来做比较好，因为有些系统内的功能需要依赖，比如注册验证、找回密码。

@sugarlesss
Copy link

那个邮件插件不能发送通知吗？我不知道，我还没有用过那个邮件插件。

通知还是交给 Core 来做比较好，因为有些系统内的功能需要依赖，比如注册验证、找回密码。

赞成，希望通知机制早日提上日程~

@JohnNiang JohnNiang mentioned this issue Jul 31, 2023
Closed
@guqing guqing mentioned this issue Aug 31, 2023
Merged
@guqing
Copy link
Member

guqing commented Aug 31, 2023

/assign

@guqing guqing mentioned this issue Sep 7, 2023
Closed
@ruibaby ruibaby mentioned this issue Sep 24, 2023
Closed
f2c-ci-robot bot pushed a commit that referenced this issue Sep 28, 2023
@guqing
feat: add the notification mechanism implementation (#4527)
9454f44 
#### What type of PR is this?
/kind feature
/milestone 2.10.x
/area core

#### What this PR does / why we need it:
新增消息和通知机制的实现

how to test it?
1. 执行以下命令配置发件服务
```shell
curl -u admin:admin -X POST 'http://localhost:8090/apis/api.console.halo.run/v1alpha1/notifiers/default-email-notifier/senderConfig' \
--header 'Content-Type: application/json' \
--data-raw '{
    "displayName": "Halo Team",
    "username": "{发件使用的邮箱}",
    "password": "{发件邮箱密码}",
    "host": "smtp.exmail.qq.com",
    "port": "587"
}'
```
2. 评论文章或页面可以收到通知
3. 文章/页面作者是评论者不发送新评论通知，回复者是评论作者不发送回复通知

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

#### Does this PR introduce a user-facing change?
```release-note
新增消息和通知机制的实现
```
@ruibaby ruibaby modified the milestones: Backlog, 2.10.0 Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@guqing guqing

Labels
area/console Issues or PRs related to the Halo Console 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.10.0
Development

Successfully merging a pull request may close this issue.

feat: add the notification mechanism implementation guqing/halo
5 participants
@ruibaby @LIlGG @ql533cocou @guqing @sugarlesss