プロジェクト

全般

プロフィール

提案 #1298

完了

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

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

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:

説明

現状の: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を使ってコマンド名を作るようにします。


ファイル

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

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

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

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

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

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

cob odo さんが約6年前に更新

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

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

cob odo さんが約6年前に更新

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

mergeしてpushしました。

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