プロジェクト

全般

プロフィール

提案 #1297

完了

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

cob odo さんが5年以上前に追加. 5年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
search

説明

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

toshi_a 初音 さんが5年以上前に更新

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

良いですね。

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

cob odo さんが5年以上前に更新

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

cob odo さんが5年以上前に更新

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

Kazuki Yagi さんが5年以上前に更新

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

toshi_a 初音 さんが5年以上前に更新

  • 担当者cob odo にセット

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

ありがとうございます。

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

cob odo さんが5年以上前に更新

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

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

toshi_a 初音 さんが5年以上前に更新

あ、じゃあmergeします

toshi_a 初音 さんが5年以上前に更新

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

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

Kazuki Yagi さんが5年以上前に更新

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

cob odo さんが5年以上前に更新

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

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

toshi_a 初音 さんが5年以上前に更新

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

merged.

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