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

通知メールの同報をメール本文などに出力して、メール内で他の誰に通知されたかをわかるようにしたい #66

Open
juno-nishizaki opened this issue Oct 23, 2021 · 4 comments · May be fixed by #73

Comments

@juno-nishizaki
Copy link
Contributor

https://redmine.org/issues/33479

@juno-nishizaki juno-nishizaki changed the title 通知メールの同報をメール本文などに表示して、メール内で他の誰に通知されたかをわかるようにしたい 通知メールの同報をメール本文などに出力して、メール内で他の誰に通知されたかをわかるようにしたい Oct 23, 2021
@ishikawa999
Copy link
Contributor

ishikawa999 commented Oct 23, 2021

残タスク

  • app/models/mailer.rbを書き換えて@recipientsを作るようにしている。今はissue_add, issue_edit, document_addのみ対応している。それ以外も対応する (リマインダーとセキュリティー通知は要検討)
  • テスト(mailer.rbの各メソッドに引数を足しているので、それらに対する対応も必要)
  • 動作検証

@ishikawa999
Copy link
Contributor

@matobaa
Copy link
Member

matobaa commented Nov 20, 2021

第17回Redmineパッチ会で取り組む

いままでの経緯:

  • 設定でチェックをつけると、メールフッタに同報先を追記するようにした
  • メールフッタは、「表示形式」をカンマ区切りで列挙した。
  • 表示形式はメール文中のAsigneeeにならった。
    • リンクにしてない、メールアドレスを出していない、id表記にしてないのはメール文中にならった以上の意図はない
    • なおX-Redmine-assigneeヘッダはid表記
  • 人が多すぎると運用によってはフッタが長すぎて都合が悪いかもなので、人数制限もできるようにした
    • はみ出すと「………」になっちゃうけど、それはチケットを見に行けばいいよね

@matobaa
Copy link
Member

matobaa commented Nov 20, 2021

CCを追加すべき対象について検討:
grep self.deliver app/models/mailer.rb

http://localhost:3000/settings?tab=notifications の一覧との比較

@matobaa matobaa linked a pull request Jan 27, 2022 that will close this issue
2 tasks
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 a pull request may close this issue.

3 participants