Actions
提案 #1298
closed:aboutuserコマンドをtwitterと無関係にする
Description
現状の:aboutuser
コマンドはuser_detail_viewプラグインで提供されていて、Plugin::Twitter::User
に対してのみユーザー名を表示するようになっています。
しかし、MessageModelの要件としてuser
メソッドでDiva::Model
を返す必要があり、Diva::Model
はtitle
メソッドを持っていて適宜オーバーライドされることを利用すれば、twitterプラグインと無関係に、常に名前(らしきもの)をコマンド名に表示することができるようになります。実際、現状の:aboutuser
の表示する名前は、Plugin::Twitter::User#title
が返すものと同じです。
この提案では、:aboutuser
をcommandプラグインに移動し、常にmessages.first.user.title
を使ってコマンド名を作るようにします。
Files
Updated by toshi_a 初音 over 6 years ago
- Status changed from 分類待ち to パッチ適用待ち
- Target version changed from 3.7 to 3.9
aboutuserコマンドはuserフィールドの値に対してIntentを発行するだけなので、別にTwitterに限る必要はないですね。
機能追加となるので3.9にします。
Updated by toshi_a 初音 over 6 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- Assignee set to cob odo
- ブランチ set to topic/1298-remove-dependency-on-twitter-from-aboutuser
パッチしてみました。確認したら develop
にmergeお願いします。
Actions