Skip to content

[codex] Streamline loading states and animations#2112

Draft
creatix wants to merge 2 commits into
masterfrom
codex/streamline-loading-states
Draft

[codex] Streamline loading states and animations#2112
creatix wants to merge 2 commits into
masterfrom
codex/streamline-loading-states

Conversation

@creatix
Copy link
Copy Markdown
Contributor

@creatix creatix commented Jun 1, 2026

Summary

  • Refactors loading state rendering and animation behavior across tabs, wallet cards, and count-up components.
  • Threads loading signals through savings/card hooks so UI surfaces can render transitional states more consistently.
  • Carries the existing package-lock cleanup included in the local commit.

Validation

  • npm run format:check (fails: Prettier reports style issues in 20 files; the check did not modify the working tree).

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-app Ignored Ignored Preview Jun 2, 2026 6:59am
solid-app-staging Ignored Ignored Preview Jun 2, 2026 6:59am

Request Review

@creatix creatix requested a review from MusabShakeel576 June 1, 2026 13:43
Commit 7270042 dropped 11 nested zod@3.25.76 entries from
package-lock.json (under @reown/appkit-*, @walletconnect/*, etc.) as
"unused", but they aren't — they're required peer-deps that npm's lock
file resolver records.

Since 7270042 only touched JS, no fresh native iOS build ran (the EAS
fingerprint stayed the same and the workflow reused the cached build).
PR #2093 added a file under patches/, bumping the fingerprint and
triggering the first fresh build since May 20. That build now fails at
"npm ci --include=dev" with "Missing: zod@3.25.76 from lock file".

Regenerate with "npm install --package-lock-only" to put the entries
back. Purely additive diff; no package versions change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants