Project

General

Profile

Actions

提案 #1297

closed

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

Added by cob odo over 6 years ago. Updated about 6 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
search

Description

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


Files

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

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に依存しませんからね。

Actions #2

Updated by cob odo over 6 years ago

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

Actions #3

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を使った検索ができる。
Actions #4

Updated by Kazuki Yagi about 6 years ago

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

Actions #5

Updated by toshi_a 初音 about 6 years ago

  • Assignee set to cob odo

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

ありがとうございます。

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

Actions #6

Updated by cob odo about 6 years ago

  • Status changed from レビュー待ち to マージ待ち

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

Actions #7

Updated by toshi_a 初音 about 6 years ago

あ、じゃあmergeします

Actions #8

Updated by toshi_a 初音 about 6 years ago

  • Status changed from マージ待ち to まだダメ

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

Actions #9

Updated by Kazuki Yagi about 6 years ago

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

Actions #10

Updated by cob odo about 6 years ago

  • Status changed from まだダメ to レビュー待ち
  • Assignee changed from cob odo to toshi_a 初音

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

Actions #11

Updated by toshi_a 初音 about 6 years ago

  • Status changed from レビュー待ち to 終了

merged.

Actions

Also available in: Atom PDF