Stop to use hard code for event info

cloudnativedaysjp / dreamkast-ui

MIT License

7 stars 2 forks source link

Stop to use hard code for event info #199

Closed takaishi closed 2 years ago

takaishi commented 2 years ago

各所でハードコードしていた cndt2021 について、APIで取得したEventを使うように変更
Boothは使わなくなったのでついでに削除

fix https://github.com/cloudnativedaysjp/dreamkast-ui/issues/192

dreamkast-cloudnativedays commented 2 years ago

Review app

https://dreamkast-ui-199.dev.cloudnativedays.jp

takaishi commented 2 years ago

index.tsxの const eventAbbr = 'cndt2021' を_app.tsx内の background-imageのURL指定に使ってDRYにしたいのだがどうやるのがいいのかわからない。React-Router使えば可変にできるのだろうか？

https://github.com/cloudnativedaysjp/dreamkast-ui/blob/83cf341117457336506e3637a64dd41409419973/src/pages/_app.tsx#L24

takaishi commented 2 years ago

@maktak1995 https://github.com/cloudnativedaysjp/dreamkast-ui/pull/199#issuecomment-1002876564 について何か方法があれば知りたいです！

maktak1995 commented 2 years ago

@takaishi eventAbbr は多分開発が進んでページが増えたりした場合にdreamcast-ui全体で参照したくなってくることが想像できるので、グローバル変数としてしまうのがいいと思います。useContext を使うのがいいかと！(参考) 今回のケースでは /utilあたりにContextを作成してあげるとよいかと思います。