Closed RamRamez closed 5 days ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
giveth-dapps-v2 | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 1, 2024 11:32am |
The firstLoading
state variable was removed from the Onboarding.view.tsx
file. This state variable was previously used to prevent flickering during the initial loading phase. The associated logic and conditional checks related to firstLoading
have also been eliminated to streamline the rendering logic.
Files | Change Summary |
---|---|
src/components/views/onboarding/Onboarding.view.tsx |
Removed the firstLoading state and adjusted related logic accordingly. |
In code’s realm, a change was made,
The flicker’s ghost will now evade.
No state to hold the first-load's sway,
A smoother path for all to play.
With lines refined and logic clear,
The onboarding cheer is ever near. 🌟
src/components/views/onboarding/Onboarding.view.tsx
[error] 53-59: This else clause can be omitted because previous branches break early. Unsafe fix: Omit the else clause. (lint/style/noUselessElse) --- [error] 55-59: This else clause can be omitted because previous branches break early. (lint/style/noUselessElse) --- [error] 57-59: This else clause can be omitted because previous branches break early. (lint/style/noUselessElse)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
https://github.com/Giveth/giveth-dapps-v2/issues/4359
Summary by CodeRabbit