-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Enhance poll #4409
Enhance poll #4409
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
別件バグなんですが、以下の既存のバグにひかかっちゃうみたいです
note.to, note.cc が 非配列(string) や undefined だと落ちる
https://github.com/syuilo/misskey/blob/e947dbac70bfc87d04cbc65af50e88d4fb67d886/src/remote/activitypub/models/note.ts#L77-L78
note.attachment が 非配列(object) や undefined だと落ちる
https://github.com/syuilo/misskey/blob/e947dbac70bfc87d04cbc65af50e88d4fb67d886/src/remote/activitypub/models/note.ts#L98-L99
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
* wip * 形式をMastodonと合わせた
もうやること全部やったかしら? |
全体的にはこれで大丈夫そう |
なんかMastodonに投票出来なくてはまってるわ |
deliverあたりにlog出すように仕込んでおいた気がするので、 |
送ってるactorが違ってた |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
お二方ありがとうございました & お疲れ様でした ✌️ |
Summary
Resolves #3507
Resolves #4407
Resolves #4408
Related to #3940
Todo