Add webhook email notifications and user email support - #6
Merged
Conversation
Introduce email notification via webhook and sync user emails from OIDC. Add src/notify.js (webhook sender + notifyApproved/notifyRejected), call notifications in admin request approve/reject, and include email in admin user list. Persist users.email in DB (create column + migration in src/db.js) and sync/insert email in OIDC callback (src/auth.js). Expose email in frontend types and UserAdmin view. Update docs (README.md, CLAUDE.md) and CI workflows (use claude-sonnet-5). Notifications are optional (MAIL_WEBHOOK_URL/TOKEN) and failures are logged without blocking flows.
|
🎉 感谢 @ZhuYuxuan9302 提交 Pull Request! 我们会尽快审查您的代码。在此期间,请确保:
🤖 我们的 MCJPG Agent 将自动对您的代码进行初步审查。 如有代码辅助需求,请在评论区 |
|
❤️ 感谢来自 @ZhuYuxuan9302 的 PR ❤️ 如果您也对 MCJPG 组织的开发工作感兴趣,欢迎联系并加入我们! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Introduce email notification via webhook and sync user emails from OIDC. Add src/notify.js (webhook sender + notifyApproved/notifyRejected), call notifications in admin request approve/reject, and include email in admin user list. Persist users.email in DB (create column + migration in src/db.js) and sync/insert email in OIDC callback (src/auth.js). Expose email in frontend types and UserAdmin view. Update docs (README.md, CLAUDE.md) and CI workflows (use claude-sonnet-5). Notifications are optional (MAIL_WEBHOOK_URL/TOKEN) and failures are logged without blocking flows.