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

评论通知问题 #67

Open
chengzhongxue opened this issue Dec 24, 2023 · 4 comments · May be fixed by #106
Open

评论通知问题 #67

chengzhongxue opened this issue Dec 24, 2023 · 4 comments · May be fixed by #106
Assignees
@guqing

Comments

@chengzhongxue
Copy link
Contributor

会给每个带有瞬间权限的用户发一条通知
image
@guqing
Copy link
Member

guqing commented May 23, 2024

如何复现这个问题呢，我无法复现
/triage needs-information

@f2c-ci-robot f2c-ci-robot bot added the triage/needs-information Indicates an issue needs more information in order to work on it. label May 23, 2024
@chengzhongxue
Copy link
Contributor Author

这个应该是瞬间更新多用户之前的版本有这个问题，后面的我还没试过，晚点我试试

@guqing
Copy link
Member

guqing commented May 23, 2024

这个应该是瞬间更新多用户之前的版本有这个问题，后面的我还没试过，晚点我试试

那我知道了，确实有这个问题，因为之前没有多用户，所以瞬间的评论事件订阅是订阅所有瞬间，如果有多人发布瞬间确实会出现此问题
/remove-triage needs-information

@f2c-ci-robot f2c-ci-robot bot removed the triage/needs-information Indicates an issue needs more information in order to work on it. label May 23, 2024
@guqing
Copy link
Member

guqing commented May 23, 2024

/assign

@guqing guqing linked a pull request May 24, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@guqing guqing

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor: adapt subscription for notifications based on expressions halo-sigs/plugin-moments
2 participants
@guqing @chengzhongxue