Skip to content

Conversation

lukaw3d
Copy link
Member

@lukaw3d lukaw3d commented Jun 6, 2025

Fixes #1863

@lukaw3d lukaw3d requested a review from buberdds June 6, 2025 18:45
Copy link

github-actions bot commented Jun 6, 2025

Deployed to Cloudflare Pages

Latest commit: d2609d8b8f7fee81a297c18fd38a13b072b903dc
Status:✅ Deploy successful!
Preview URL: https://beea57f5.oasis-wallet.pages.dev
Alias: https://pr-2188.oasis-wallet.pages.dev

@lukaw3d lukaw3d force-pushed the lw/flashing-wrong-layout branch from a89f18e to d2609d8 Compare June 6, 2025 21:40
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​jest/​fake-timers@​29.7.01001006796100

View full report

@lukaw3d
Copy link
Member Author

lukaw3d commented Jun 6, 2025

playwright test broke because useRouteRedirects now redirects away from e2e before setIsResponsiveContextInitialized(true) lets it to be rendered. Ignoring this until #823

@buberdds
Copy link
Contributor

buberdds commented Jun 9, 2025

What's an issue with a failing test? It's fine when I run it locally using this side branch.

edit: oh I did not see your comment. So the plan is to merge this with a test that will always fail on CI?

@lukaw3d lukaw3d marked this pull request as draft June 9, 2025 15:12
@lukaw3d
Copy link
Member Author

lukaw3d commented Jun 9, 2025

I would just ignore this PR and layout issue until #823

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.

Extension: app loading state flashes wrong layout
2 participants