search

w3c / wot-jp-cg

W3C Web of Things Japanese Community Group
https://wot-jp-cg.netlify.app/
Other
9 stars 10 forks source link

Algolia DocSearch 導入 #87

Closed hidessy closed 2 years ago

hidessy commented 2 years ago

82 で議論されてます、Algolia DocSearchを適応いたしました。

検索バーより検索が可能になります。

よろしくお願いいたします。

netlify[bot] commented 2 years ago

Deploy Preview for wot-jp-cg-docusaurus canceled.

Name Link
Latest commit 9c5c12206102c6ea8009ab80e92ae4888bae7516
Latest deploy log https://app.netlify.com/sites/wot-jp-cg-docusaurus/deploys/62ac2fc777dcfd0008663273
dynamis commented 2 years ago

Netlifyのデプロイ失敗は対象コンテンツの変更がないと去れてるからですが設定ファイルの変更だけでは検知されないだけ、実際にビルドプロセス走らせれば大丈夫という想定ですね。

k-toumura commented 2 years ago

PRありがとうございます。 netlifyでpreviewができないということだと思うので、まずはすぐにmergeして、うまくいかなかったら戻す、とした方がよさそうですね。

念のためもう20分ほど待って、18:00になったらマージしてみます。

dajiaji commented 2 years ago

ちょっと全然わかっておらずチラ見でのコメントですが、apiKeyって公開で大丈夫なのですか？ github settings > actions secrets あたりから環境変数で流し込む必要は無いですか？（変なこと言っていたらすみません）

dynamis commented 2 years ago

apiKeyって公開で大丈夫なのですか？

より厳密には Search Only API Key "This is the public API key to use in your frontend code. This key is only usable for search queries and sending data to the Insights API." なので大丈夫なのです。

いずれにしてもブラウザから問いあわせする時に使う = JS 側に公開されるキーです。

Admin/Usage/Monitoring API Key というのがそれぞれ別にあり、それらは公開禁止という使い分けです。

dajiaji commented 2 years ago

了解です。安心しました。

dynamis commented 2 years ago

Netlifyのデプロイ失敗は対象コンテンツの変更がないと去れてるからですが設定ファイルの変更だけでは検知されないだけ、実際にビルドプロセス走らせれば大丈夫という想定ですね。

netlifyでpreviewができないということだと思うので、まずはすぐにmergeして、うまくいかなかったら戻す、とした方がよさそうですね。

すみません、私が試験的に別のデプロイをしていた方が残ってて fail していただけだったかもです。

https://deploy-preview-87--wot-jp-cg.netlify.app/ で試して頂けます。

設定のせいか日本語に最適化されている気はしないが日本語での検索も一応できるのでひとまずは大丈夫かな、クローラ側の設定で改善するのかはちょっと調べて後日試して見る、想定です。

k-toumura commented 2 years ago

私もよくわからず、改めて調べました。下記のページに@dynamis さんが書いていた内容がまとまってますね。 https://www.algolia.com/doc/guides/security/api-keys/#search-only-api-key

プレビューもできるようなので、マージは来週火曜日の午後あたりまで待ってみます。

k-toumura commented 2 years ago

特にコメントなどありませんでしたので、マージいたします。