Actions
提案 #1297
closedsearchプラグインからtwitterプラグインへの依存を除去
プラグイン名:
search
Description
search機能がspell化されたり、search spellがありさえすればtwitterアカウントが登録されていなくても検索タブが表示されたりする #1291 昨今です。
そこで、twitterプラグイン自体を外したとしても、searchプラグインを使えるようにするパッチを書きました。
Files
Updated by toshi_a 初音 over 6 years ago
- Status changed from 分類待ち to パッチ適用待ち
- Target version changed from 3.7 to 3.8
良いですね。
intentブロックは、twitter_searchプラグインを新設してそこに書こうと思います。TwitterプラグインはIntentに依存しませんからね。
Updated by cob odo over 6 years ago
了解です。twitterプラグインはscoreプラグインを通じて間接的にintentプラグインに依存していますが、明示的にしておいたほうがいいと思います。
Updated by cob odo over 6 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- Target version changed from 3.8 to 3.9
- ブランチ set to topic/1297-search-without-twitter
twitter_searchプラグインを新設する形で topic/1297-search-without-twitter ブランチにpushしました。派生元は( #1298 に準じて)developにしてあります。
この修正では、以下の2つを確認する必要があると思いますが、レビューはどうしましょうね。一応、自分では確認していますけども。
- twitterプラグインがあり、twitterアカウントが登録された状態で、twitter検索ができることが維持されている。
- twitterプラグインが無い状態で、別worldでもsearch spellを使った検索ができる。
Updated by toshi_a 初音 about 6 years ago
- Assignee set to cob odo
Updated by toshi_a 初音 about 6 years ago
- Status changed from マージ待ち to まだダメ
パッチをもう一度確認したところ、/run.shというファイルが入っていました。
一度ブランチは削除しましたが、同じ名前のブランチに上記ファイルを削ったcommitを行ったので、再度確認してください。
Updated by Kazuki Yagi about 6 years ago
最新commitに変更後、起動確認した所、下に関しては確認できましたので再度報告します。
Updated by cob odo about 6 years ago
- Status changed from まだダメ to レビュー待ち
- Assignee changed from cob odo to toshi_a 初音
お手数おかけしました。無関係なファイルでしたが、一応、私の方でも再度動作確認しました。
Actions