cloudnativedaysjp / dreamkast-ui

MIT License
7 stars 2 forks source link

Replace to RTK Query #305

Closed hrk091 closed 2 years ago

hrk091 commented 2 years ago

openapi clientを使っている箇所を全てRTK Queryに置き換えました。 https://github.com/cloudnativedaysjp/dreamkast-ui/pull/298 で整備した基盤を使うことで、いくつかの技術負債も一緒に返済しています。

cookieも引き続き飛ぶので、現backendに対しての後方互換性を担保しています。 (dkの方で後方互換についてコメントしましたが、よく考えたら同一ドメインの間は変わらずcookieが飛ぶので、dk-uiが先に進む分には後方互換担保が可能でした > @jacopen )

依存している https://github.com/cloudnativedaysjp/dreamkast-ui/pull/298 が in review なので、一旦そこに向けてPRを切っています。

github-actions[bot] commented 2 years ago

Review app

jacopen commented 2 years ago

一通り見ましたー、自分からは特にないのでapproveしました!

hrk091 commented 2 years ago

ありがとうございます!mergeさせていただきますー