プロジェクト

全般

プロフィール

提案 #1297

searchプラグインからtwitterプラグインへの依存を除去

cob odo23日前に追加. 14日前に更新.

ステータス:
レビュー待ち
優先度:
通常
担当者:
-
対象バージョン:
プラグイン名:
search
ブランチ:
topic/1297-search-without-twitter

説明

search機能がspell化されたり、search spellがありさえすればtwitterアカウントが登録されていなくても検索タブが表示されたりする #1291 昨今です。
そこで、twitterプラグイン自体を外したとしても、searchプラグインを使えるようにするパッチを書きました。

search-without-twitter.patch (1.79 KB) search-without-twitter.patch cob odo, 2018-09-29 19:07

関係しているリビジョン

リビジョン 6774c660 (差分)
cob odo14日前に追加

twitter_searchプラグインを作成してsearchプラグインからtwitterプラグインへの依存を除去 refs #1297

履歴

#1 toshi_a 初音22日前に更新

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

良いですね。

intentブロックは、twitter_searchプラグインを新設してそこに書こうと思います。TwitterプラグインはIntentに依存しませんからね。

#2 cob odo22日前に更新

了解です。twitterプラグインはscoreプラグインを通じて間接的にintentプラグインに依存していますが、明示的にしておいたほうがいいと思います。

#3 cob odo14日前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 対象バージョン3.8 から 3.9 に変更
  • ブランチtopic/1297-search-without-twitter にセット

twitter_searchプラグインを新設する形で topic/1297-search-without-twitter ブランチにpushしました。派生元は( #1298 に準じて)developにしてあります。

この修正では、以下の2つを確認する必要があると思いますが、レビューはどうしましょうね。一応、自分では確認していますけども。

  • twitterプラグインがあり、twitterアカウントが登録された状態で、twitter検索ができることが維持されている。
  • twitterプラグインが無い状態で、別worldでもsearch spellを使った検索ができる。

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