バグ #1219
完了
cob odo さんが6年以上前に追加.
6年以上前に更新.
説明
現在のdevelopブランチ(score merge後)で、tweet引用が全体的に展開されていないようです。例のスクリーンショットを添付します。
ファイル
- ステータス を 分類待ち から 実装待ち に変更
- 担当者 を toshi_a 初音 にセット
これ、scoreをMessage Modelから呼び出すみたいになっちゃうので良くないですね。
そんなに大変そうじゃないので、イベントフィルタで引用ツイートを取得するAPIを実装してみようかなと思います。
- ステータス を 実装待ち から レビュー待ち に変更
- 担当者 を toshi_a 初音 から cob odo に変更
- ブランチ を topic/1219-quoted-tweet にセット
バグ修正というか機能拡張みたいになりました。
Scoreに含まれるURIから、model_of_uriフィルタが反応できるModelをMessage Modelと仮定して引用Messageとして表示するようにしてみました。
Diva::Model.handleを使ってURIからModelに変換する方法を定義してあれば、どのようなMessage Modelにも対応できます。
mikutterのCK/CS凍結によりテストできなくなりました。すみませんがよろしくお願いします。
- 担当者 を cob odo から toshi_a 初音 に変更
- 関連している バグ #1226: 同一URLに対してmodel_of_uriで複数のModel Classが取得できる場合に、引用で表示する方法の選択。 を追加
過去にCK/CS凍結された経験のある電話番号を別垢に紐づけて登録したらCK/CSが取得できたので、結局テストできました。
が確認できたのでこのチケットとしてはOKだと思います。ついでに、
も確認しました。
チケットとしては不具合修正ですが、新機能でもあるので、developへのmergeは(僕の方では)控えています。
また、 #1226 はこの件を前提に書いています。併せて確認してもらえるとありがたいです。
ありがとうございます。この件はmergeしておきました。
他の形式にエクスポート: Atom
PDF