-
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
issue-28 #255
Conversation
Visit the preview URL for this PR (updated for commit 62d4388): https://cafeore-2024--pr255-issue-28-rjovmx02.web.app (expires Mon, 28 Oct 2024 18:11:10 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: b62dcde668e26b6280dd8317f3f875013f6f22f5 |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
スペースだ改行だを修正したら?とのことだったのでLGTMでcommitしました |
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.
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.
PRのタイトルだけ、他のPRと似せてわかりやすいタイトルつけてください
これ、もしかしたら早期リターンをやめて、三項演算子で要素の構造を壊さずにOutletの出し入れをしたら修正できるかも。余力あったらやってみてほしい |
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.
一旦merge
close #28
オフライン(!isOnline==true)だとOutletが表示されないようにした
もし、if文をreturnの中に入れたかったら入れてもOK