fix: support missing bridge #80

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.

Sign up for GitHub

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

Jump to bottom

Open

cpb8010 wants to merge 1 commit into main from missing-bridge

cpb8010 commented Apr 22, 2025

What 💻

Skip assigning l2 shared bridge if missing

Why ✋

Local nodes and non-rollups won't have this bridge information
This was blocking testing SSO on localnode when using smart account

Evidence 📷

tdb, works locally with test node

Notes 📝

low risk


          fix: support missing bridge

bb0d444

This fails with localnode that doesn't have any bridges

cpb8010 requested a review from a team

April 22, 2025 15:14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet