お知らせをDiscord側に通知するように

[nenohi]

What

お知らせ発行時にDiscord側に通知するように修正

Why

Additional info (optional)

Checklist

• [x] Read the contribution guide
• [x] Test working in a local environment
• [ ] (If needed) Add story of storybook
• [ ] (If needed) Update CHANGELOG.md
• [ ] (If possible) Add tests

[u1-liquid]

pnpm clean; pnpm -r build:tsc; pnpm build; pnpm build-misskey-js-with-types

走らせてmisskey-js更新してください

[sonarcloud[bot]]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[u1-liquid]

こう実装すると後々の拡張性が皆無で他の変更とかも転送したくなったりslackとかにwebhook飛ばしたくなったりしたとき全部リワークになりそうだけど本当にこれでいいんだろうかね

できれば通報のwebhook https://github.com/MisskeyIO/misskey/pull/615 同様に既存のwebhook設定に組み込みたい気持ちがある もしくは https://redirect.github.com/misskey-dev/misskey/pull/13758 のチェリーピック＆追加修正とか

[nenohi]

確かにDiscordとしか考えてないけどMessageとかも全部諸々変更になるからめんどくさそう

[nenohi]

既存のWebhook設定使ってもどこかしらで形式の変更が必要になるんじゃないかな？ Discordは

{
  content:"内容"
}

になるけど、既存のもの使うと

{
   updatedAt: null,
  title: ps.title,
  text: ps.text,
  imageUrl: ps.imageUrl,
  icon: ps.icon,
  display: ps.display,
  forExistingUsers: ps.forExistingUsers,
  needConfirmationToRead: ps.needConfirmationToRead,
  closeDuration: ps.closeDuration,
  displayOrder: ps.displayOrder,
  silence: ps.silence,
  userId: ps.userId,
}

が送信されてしまうので何かしらを噛ませないとDisocrd側に通知されなさそうな気がしてる

[u1-liquid]

webhookにtypeフィールド追加して分岐させるとかがいいんじゃないかなと

[nenohi]

Discord、Slack、Otherにして Otherだったら詳細な部分できるようにして入力させるとかです？

[u1-liquid]

そういう感じでよさそう otherよりはtailscaleみたいにnone https://tailscale.com/kb/1213/webhooks#setting-up-a-webhook-endpoint とかmisskeyって名前がいいかもしれない

[nenohi]

できれば通報のwebhook https://github.com/MisskeyIO/misskey/pull/615 同様に既存のwebhook設定に組み込みたい気持ちがある

こっちで行ったほうが修正しやすそうなのでこれで

[nenohi]

修正めんどくさいのでブランチ破壊して新しくPR建てますね