プロジェクト

全般

プロフィール

提案 #1298

:aboutuserコマンドをtwitterと無関係にする

cob odo3ヶ月前に追加. 2ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
ブランチ:
topic/1298-remove-dependency-on-twitter-from-aboutuser

説明

現状の:aboutuserコマンドはuser_detail_viewプラグインで提供されていて、Plugin::Twitter::Userに対してのみユーザー名を表示するようになっています。

しかし、MessageModelの要件としてuserメソッドでDiva::Modelを返す必要があり、Diva::Modeltitleメソッドを持っていて適宜オーバーライドされることを利用すれば、twitterプラグインと無関係に、常に名前(らしきもの)をコマンド名に表示することができるようになります。実際、現状の:aboutuserの表示する名前は、Plugin::Twitter::User#titleが返すものと同じです。

この提案では、:aboutuserをcommandプラグインに移動し、常にmessages.first.user.titleを使ってコマンド名を作るようにします。

関係しているリビジョン

リビジョン bacb1531 (差分)
cob odo2ヶ月前に追加

aboutuserコマンドをtwitter非依存化 refs #1298

履歴

#1 toshi_a 初音2ヶ月前に更新

  • ステータス分類待ち から パッチ適用待ち に変更
  • 対象バージョン3.7 から 3.9 に変更

aboutuserコマンドはuserフィールドの値に対してIntentを発行するだけなので、別にTwitterに限る必要はないですね。
機能追加となるので3.9にします。

#2 toshi_a 初音2ヶ月前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 担当者cob odo にセット
  • ブランチtopic/1298-remove-dependency-on-twitter-from-aboutuser にセット

パッチしてみました。確認したら develop にmergeお願いします。

#3 cob odo2ヶ月前に更新

  • ステータスレビュー待ち から マージ待ち に変更

ありがとうございます。OKでした。

#4 cob odo2ヶ月前に更新

  • ステータスマージ待ち から 終了 に変更

mergeしてpushしました。

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