提案 #1297
完了
searchプラグインからtwitterプラグインへの依存を除去
cob odo さんが約6年前に追加.
ほぼ6年前に更新.
説明
search機能がspell化されたり、search spellがありさえすればtwitterアカウントが登録されていなくても検索タブが表示されたりする #1291 昨今です。
そこで、twitterプラグイン自体を外したとしても、searchプラグインを使えるようにするパッチを書きました。
ファイル
- ステータス を 分類待ち から パッチ適用待ち に変更
- 対象バージョン を 3.7 から 3.8 に変更
良いですね。
intentブロックは、twitter_searchプラグインを新設してそこに書こうと思います。TwitterプラグインはIntentに依存しませんからね。
了解です。twitterプラグインはscoreプラグインを通じて間接的にintentプラグインに依存していますが、明示的にしておいたほうがいいと思います。
- ステータス を パッチ適用待ち から レビュー待ち に変更
- 対象バージョン を 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を使った検索ができる。
パッチを確認した所、下に関しては確認できましたので報告します。
パッチを確認した所、下に関しては確認できましたので報告します。
ありがとうございます。
cob odo
大きな影響がある変更ではないので、このまま「マージ待ち」にしてしまって構いません。
- ステータス を レビュー待ち から マージ待ち に変更
- ステータス を マージ待ち から まだダメ に変更
パッチをもう一度確認したところ、/run.shというファイルが入っていました。
一度ブランチは削除しましたが、同じ名前のブランチに上記ファイルを削ったcommitを行ったので、再度確認してください。
最新commitに変更後、起動確認した所、下に関しては確認できましたので再度報告します。
- ステータス を まだダメ から レビュー待ち に変更
- 担当者 を cob odo から toshi_a 初音 に変更
お手数おかけしました。無関係なファイルでしたが、一応、私の方でも再度動作確認しました。
他の形式にエクスポート: Atom
PDF
