Actions
バグ #1219
closed
Tweet内のTweet引用が展開されていない
プラグイン名:
twitter, score
クラッシュする:
No
Description
現在のdevelopブランチ（score merge後）で、tweet引用が全体的に展開されていないようです。例のスクリーンショットを添付します。
Files
Related issues
Updated by toshi_a 初音 over 6 years ago
- Status changed from 分類待ち to 実装待ち
- Assignee set to toshi_a 初音
確かに表示されてないですね
Updated by toshi_a 初音 over 6 years ago
これ、scoreをMessage Modelから呼び出すみたいになっちゃうので良くないですね。
そんなに大変そうじゃないので、イベントフィルタで引用ツイートを取得するAPIを実装してみようかなと思います。
Updated by toshi_a 初音 over 6 years ago
- Status changed from 実装待ち to レビュー待ち
- Assignee changed from toshi_a 初音 to cob odo
- ブランチ set to topic/1219-quoted-tweet
バグ修正というか機能拡張みたいになりました。
Scoreに含まれるURIから、model_of_uriフィルタが反応できるModelをMessage Modelと仮定して引用Messageとして表示するようにしてみました。
Diva::Model.handleを使ってURIからModelに変換する方法を定義してあれば、どのようなMessage Modelにも対応できます。
Updated by cob odo over 6 years ago
mikutterのCK/CS凍結によりテストできなくなりました。すみませんがよろしくお願いします。
Updated by cob odo over 6 years ago
- Related to バグ #1226: 同一URLに対してmodel_of_uriで複数のModel Classが取得できる場合に、引用で表示する方法の選択。 added
Updated by cob odo over 6 years ago
過去にCK/CS凍結された経験のある電話番号を別垢に紐づけて登録したらCK/CSが取得できたので、結局テストできました。
- Tweet内のTweet引用
が確認できたのでこのチケットとしてはOKだと思います。ついでに、
- Tweet内のToot引用（Worldonにhandle付加）
- Tweet内のOpenGraphProtocol/TwitterCardによる引用（ https://github.com/cobodo/mikutter-twitter-card を新たに作成）
も確認しました。
Updated by cob odo over 6 years ago
チケットとしては不具合修正ですが、新機能でもあるので、developへのmergeは（僕の方では）控えています。
また、 #1226 はこの件を前提に書いています。併せて確認してもらえるとありがたいです。
Updated by toshi_a 初音 over 6 years ago
- Status changed from レビュー待ち to 終了
ありがとうございます。この件はmergeしておきました。
Actions