プロジェクト

全般

プロフィール

バグ #1219

完了

Tweet内のTweet引用が展開されていない

cob odo さんが6年以上前に追加. 6年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
twitter, score
クラッシュする:
いいえ

説明

現在のdevelopブランチ(score merge後)で、tweet引用が全体的に展開されていないようです。例のスクリーンショットを添付します。


ファイル


関連するチケット

関連している バグ #1226: 同一URLに対してmodel_of_uriで複数のModel Classが取得できる場合に、引用で表示する方法の選択。実装待ち操作

toshi_a 初音 さんが6年以上前に更新

  • ステータス分類待ち から 実装待ち に変更
  • 担当者toshi_a 初音 にセット

確かに表示されてないですね

toshi_a 初音 さんが6年以上前に更新

これ、scoreをMessage Modelから呼び出すみたいになっちゃうので良くないですね。
そんなに大変そうじゃないので、イベントフィルタで引用ツイートを取得するAPIを実装してみようかなと思います。

toshi_a 初音 さんが6年以上前に更新

  • ステータス実装待ち から レビュー待ち に変更
  • 担当者toshi_a 初音 から cob odo に変更
  • ブランチtopic/1219-quoted-tweet にセット

バグ修正というか機能拡張みたいになりました。

Scoreに含まれるURIから、model_of_uriフィルタが反応できるModelをMessage Modelと仮定して引用Messageとして表示するようにしてみました。
Diva::Model.handleを使ってURIからModelに変換する方法を定義してあれば、どのようなMessage Modelにも対応できます。

cob odo さんが6年以上前に更新

mikutterのCK/CS凍結によりテストできなくなりました。すみませんがよろしくお願いします。

cob odo さんが6年以上前に更新

  • 担当者cob odo から toshi_a 初音 に変更

cob odo さんが6年以上前に更新

  • 関連している バグ #1226: 同一URLに対してmodel_of_uriで複数のModel Classが取得できる場合に、引用で表示する方法の選択。 を追加

cob odo さんが6年以上前に更新

過去にCK/CS凍結された経験のある電話番号を別垢に紐づけて登録したらCK/CSが取得できたので、結局テストできました。

  • Tweet内のTweet引用

が確認できたのでこのチケットとしてはOKだと思います。ついでに、

も確認しました。

cob odo さんが6年以上前に更新

チケットとしては不具合修正ですが、新機能でもあるので、developへのmergeは(僕の方では)控えています。

また、 #1226 はこの件を前提に書いています。併せて確認してもらえるとありがたいです。

toshi_a 初音 さんが6年以上前に更新

  • ステータスレビュー待ち から 終了 に変更

ありがとうございます。この件はmergeしておきました。

他の形式にエクスポート: Atom PDF