Project

General

Profile

バグ #1219

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

Added by cob odo over 1 year ago. Updated over 1 year ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
twitter, score
ブランチ:
topic/1219-quoted-tweet
クラッシュする:
No

Description

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


Files


Related issues

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

Associated revisions

Revision 0be9a6fc (diff)
Added by toshi_a 初音 over 1 year ago

sub_parts_quote: 引用されているMessageを、Scoreを利用して取得するようにした refs #1219

History

#1

Updated by toshi_a 初音 over 1 year ago

  • Status changed from 分類待ち to 実装待ち
  • Assignee set to toshi_a 初音

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

#2

Updated by toshi_a 初音 over 1 year ago

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

#3

Updated by toshi_a 初音 over 1 year 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にも対応できます。

#4

Updated by cob odo over 1 year ago

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

#5

Updated by cob odo over 1 year ago

  • Assignee changed from cob odo to toshi_a 初音
#6

Updated by cob odo over 1 year ago

  • Related to バグ #1226: 同一URLに対してmodel_of_uriで複数のModel Classが取得できる場合に、引用で表示する方法の選択。 added
#7

Updated by cob odo over 1 year ago

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

  • Tweet内のTweet引用

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

も確認しました。

#8

Updated by cob odo over 1 year ago

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

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

#9

Updated by toshi_a 初音 over 1 year ago

  • Status changed from レビュー待ち to 終了

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

Also available in: Atom PDF