-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP]loadingの挙動を統一する #136
base: develop
Are you sure you want to change the base?
[WIP]loadingの挙動を統一する #136
Conversation
Deploying sos24-client with Cloudflare Pages
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
何も返さないLoadingコンポーネントを作ってもらってローディング時はそちらを返すようにしてもらってもいいですか?
あと、arataさんがDiscordでおっしゃてた通りの順番でマージする&メインブランチが変更されてたのでそれをこのブランチにマージして再チェックした方がいいかも |
ちょっと怖いのでもし対面のタイミングがあったら手伝ってもらえませんか... |
(appareくんから特に言いたいこととかがなければ)今日の夜とかにでも僕が対面で引き継げます 👍 |
Co-authored-by: Takaharu Nakamura <[email protected]>
Co-authored-by: Takaharu Nakamura <[email protected]>
Co-authored-by: Takaharu Nakamura <[email protected]>
Co-authored-by: Takaharu Nakamura <[email protected]>
見落としていました |
メモ: マージ時にベースブランチをdevelopに変更 |
あとでSWRからのデータのとり方の修正をします |
スケルトンを使うべきかもという話が上がっていますがどうしましょうか |
close #112
close #56
とりあえずfalsyな値を使ってloadingを検知していた箇所をisLoadingを使うようにしました。
isLoadingの場合は何もないを返すようにしました。