Enable to show unlisted tagged toots on the hashtag timeline#32
Enable to show unlisted tagged toots on the hashtag timeline#32takayamaki merged 3 commits intoimas:imastodonfrom
Conversation
takayamaki
left a comment
takayamaki
left a comment
There was a problem hiding this comment.
LGTM
PRの規模的に念のため #34 を先にマージしてから取り込むつもり
deflis
left a comment
deflis
left a comment
There was a problem hiding this comment.
実際の確認まだしてないですが、懸念がありそうな雰囲気があるので修正依頼にしておきます。
| end | ||
|
|
||
| return if status.account.silenced? || !status.public_visibility? || status.reblog? | ||
| return if status.account.silenced? || !status.public_visibility? && !status.unlisted_visibility? || status.reblog? |
There was a problem hiding this comment.
この下のif文も修正しないと、deliver_to_publicにunlistedも行かないですかね？
There was a problem hiding this comment.
あー書き忘れてましたね()
直します
あとrender_anonymous_payload(status)ってこのままで大丈夫ですかね
There was a problem hiding this comment.
unlistedなのにLTLまで流れて行ってしまうバグを見落とすとかあまりにもガバガバだったので恥じ入るばかり
render_anonymous_payloadはnodeにredis pub/sub経由でぶん投げるメッセージを生成しているだけなので問題なさそう
There was a problem hiding this comment.
ハッシュタグの付いたリプライはハッシュタグに乗るっぽいけど元からの仕様っぽいので大丈夫そう
|
記憶が1分も持たないことが判明しました（直しました |
takayamaki
left a comment
takayamaki
left a comment
There was a problem hiding this comment.
テスト用に使ってる南東タグのリンクをクリックすると初期表示用に/api/v1/timelines/tag/南東が呼ばれるのだけれど、その中で思いっきりdirectとprivateが漏れてる
|
修正しました。自環境で確認したところ大丈夫でした。 |
|
LGTM |
|
というわけで #34 をマージした後でこっちも一緒に取り込みまーす |
imas#32 * Enable to show unlisted tagged toots on the hashtag timeline * Add missing conditional expression * Correcting visibility filtering
3 participants
Based on #29
Currently, only "public" toots with hashtags are flowing to the hashtag timeline (/api/v1/streaming/hashtag), but this modifying enables to flow "unlisted" toots to the hash tag timeline.
現在、ハッシュタグ付きのトゥートについてはpublicなもののみがハッシュタグタイムライン(/api/v1/streaming/hashtag)に流れていますが、unlistedなトゥートでもハッシュタグタイムラインに流れるようにします。