提案 #1297
closed
searchプラグインからtwitterプラグインへの依存を除去
Added by cob odo almost 8 years ago.
Updated over 7 years ago.
Description
search機能がspell化されたり、search spellがありさえすればtwitterアカウントが登録されていなくても検索タブが表示されたりする #1291 昨今です。
そこで、twitterプラグイン自体を外したとしても、searchプラグインを使えるようにするパッチを書きました。
Files
- Status changed from 分類待ち to パッチ適用待ち
- Target version changed from 3.7 to 3.8
良いですね。
intentブロックは、twitter_searchプラグインを新設してそこに書こうと思います。TwitterプラグインはIntentに依存しませんからね。
了解です。twitterプラグインはscoreプラグインを通じて間接的にintentプラグインに依存していますが、明示的にしておいたほうがいいと思います。
- 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を使った検索ができる。
パッチを確認した所、下に関しては確認できましたので報告します。
パッチを確認した所、下に関しては確認できましたので報告します。
ありがとうございます。
@cob odo
大きな影響がある変更ではないので、このまま「マージ待ち」にしてしまって構いません。
- Status changed from レビュー待ち to マージ待ち
- Status changed from マージ待ち to まだダメ
パッチをもう一度確認したところ、/run.shというファイルが入っていました。
一度ブランチは削除しましたが、同じ名前のブランチに上記ファイルを削ったcommitを行ったので、再度確認してください。
最新commitに変更後、起動確認した所、下に関しては確認できましたので再度報告します。
- Status changed from まだダメ to レビュー待ち
- Assignee changed from cob odo to toshi_a 初音
お手数おかけしました。無関係なファイルでしたが、一応、私の方でも再度動作確認しました。
- Status changed from レビュー待ち to 終了
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by