Auth0 RTK Query integration

cloudnativedaysjp / dreamkast-ui

MIT License

7 stars 2 forks source link

Closed hrk091 closed 2 years ago

hrk091 commented 2 years ago

以下を実装しました

一旦draft PRとして作りました。まだハードコード部分が残っていたりいくつか残課題があるので、そちらの対処が終わったらdraft解除します。

github-actions[bot] commented 2 years ago

Review app

hrk091 commented 2 years ago

auth0のパラメータやBASE_URL/API_BASE_URLを環境変数から取得するようにし、ハードコードを消しました。 Frontendに埋め込む環境変数は全て公開して問題ない値だけなので、local dev用に .env.developmentも追加しています。

@maktak1995 @kahirokunn 課題が解消できたので、draft解除します。reviewお願いします。

hrk091 commented 2 years ago

CIがコケてるのは、build時に環境変数が必要になったことでbuildが失敗しているからだけなので、reviewしてもらって大丈夫です。 CIもあとで直しておきます。

hrk091 commented 2 years ago

自ホストのbaseUrlの解決などをSPAに倒してやるようにしたのですが、

という現方式にマッチしているアプローチとは言い難いので、やはりSSRで環境変数周りを設定するようにし、container実行時の環境変数で受けられるように修正します。まだcommit積みますので、reviewはその後で大丈夫です

hrk091 commented 2 years ago

勘違いしてた。NEXT_PUBLIC_XXXはbuild時に埋め込むわけじゃなく、SSRしているnode.jsの実行環境で指定するだけでexposeされるのか。それなら余計なことしなくて良さそう（SPAの開発経験が多いので、どうしてもSPA的に考えてしまう。。。）