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

BotからBotのフォロー・リプライができないように #678

Merged
merged 4 commits into from
Aug 7, 2024

Conversation

nenohi
Copy link
Member

@nenohi nenohi commented Jul 29, 2024

What

Why

Additional info (optional)

Checklist

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

@nenohi
Copy link
Member Author

nenohi commented Jul 29, 2024

何もテストしてません(懺悔

@u1-liquid
Copy link
Member

動作確認はしてもろて…

@nenohi
Copy link
Member Author

nenohi commented Jul 29, 2024

テストコード書いたら許される?

@u1-liquid
Copy link
Member

別にローカルで手動で触って動作確認できてたらいいと思う

@nenohi
Copy link
Member Author

nenohi commented Jul 29, 2024

フォロー時にフロント何も変更してないからエラー画面とか出ないけどいい・・・?
(多分これ画面出さないのは外部サーバーとか向けに投げたときかも)
BotToBotのリプライも普通に通ってしまったので修正入れるか~~~

@u1-liquid
Copy link
Member

api上ではエラー出てるけどクライアントにエラーハンドリングがないって話なら一旦そのままでいいと思う

@nenohi
Copy link
Member Author

nenohi commented Jul 29, 2024

もうちょっと良い実装方法ありそうだけど一旦エラーになった

@u1-liquid
Copy link
Member

BotToBotよりいい表現ないのかな
(単純にボットからボットに何かをするのが禁止って言うより自動化されたアカウント同士の無限ループみたいなのを防ぐため~みたいなニュアンスになるものがほしい)

@u1-liquid
Copy link
Member

build-misskey-js-with-types 回す必要がある

Copy link

sonarcloud bot commented Aug 7, 2024

@u1-liquid
Copy link
Member

エラーコードってtypesに入らないんだっけ・・・?まあいっか

@u1-liquid u1-liquid merged commit 50066cd into MisskeyIO:io Aug 7, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants