プロジェクト

全般

プロフィール

提案 #1297

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

cob odo3ヶ月前に追加. 22日前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
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 odo2ヶ月前に追加

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

リビジョン 9e0f3cc9 (差分)
cob odo約2ヶ月前に追加

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

履歴

#1 toshi_a 初音3ヶ月前に更新

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

良いですね。

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

#2 cob odo3ヶ月前に更新

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

#3 cob odo2ヶ月前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 対象バージョン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を使った検索ができる。

#4 Kazuki Yagi約2ヶ月前に更新

パッチを確認した所、下に関しては確認できましたので報告します。

#5 toshi_a 初音約2ヶ月前に更新

  • 担当者cob odo にセット

パッチを確認した所、下に関しては確認できましたので報告します。

ありがとうございます。

cob odo
大きな影響がある変更ではないので、このまま「マージ待ち」にしてしまって構いません。

#6 cob odo約2ヶ月前に更新

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

ありがとうございます。マージ待ちにします。

#7 toshi_a 初音約2ヶ月前に更新

あ、じゃあmergeします

#8 toshi_a 初音約2ヶ月前に更新

  • ステータスマージ待ち から まだダメ に変更

パッチをもう一度確認したところ、/run.shというファイルが入っていました。
一度ブランチは削除しましたが、同じ名前のブランチに上記ファイルを削ったcommitを行ったので、再度確認してください。

#9 Kazuki Yagi約2ヶ月前に更新

最新commitに変更後、起動確認した所、下に関しては確認できましたので再度報告します。

#10 cob odo24日前に更新

  • ステータスまだダメ から レビュー待ち に変更
  • 担当者cob odo から toshi_a 初音 に変更

お手数おかけしました。無関係なファイルでしたが、一応、私の方でも再度動作確認しました。

#11 toshi_a 初音22日前に更新

  • ステータスレビュー待ち から 終了 に変更

merged.

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