Project

General

Profile

提案 #1297

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

Added by cob odo 10 months ago. Updated 8 months ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
search
ブランチ:
topic/1297-search-without-twitter

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

Associated revisions

Revision 6774c660 (diff)
Added by cob odo 9 months ago

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

Revision 9e0f3cc9 (diff)
Added by cob odo 9 months ago

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

History

#1

Updated by toshi_a 初音 10 months ago

  • Status changed from 分類待ち to パッチ適用待ち
  • Target version changed from 3.7 to 3.8

良いですね。

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

#2

Updated by cob odo 10 months ago

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

#3

Updated by cob odo 9 months 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を使った検索ができる。
#4

Updated by Kazuki Yagi 9 months ago

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

#5

Updated by toshi_a 初音 9 months ago

  • Assignee set to cob odo

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

ありがとうございます。

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

#6

Updated by cob odo 9 months ago

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

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

#7

Updated by toshi_a 初音 9 months ago

あ、じゃあmergeします

#8

Updated by toshi_a 初音 9 months ago

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

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

#9

Updated by Kazuki Yagi 9 months ago

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

#10

Updated by cob odo 8 months ago

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

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

#11

Updated by toshi_a 初音 8 months ago

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

merged.

Also available in: Atom PDF