Conversation
still needs front-end config
also try to fix the server in prod
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
Visit the preview URL for this PR (updated for commit 1baca33): https://zksync-auth-server-staging--pr255-main-net-deploy-0hgqtlzz.web.app (expires Thu, 22 Jan 2026 15:51:38 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 509a9c9ea42583076f531c53cf2979c544d5d0b7 |
breaks page loading
in script comment
tested locally
have issues with github review requirements
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jackpooleyml
left a comment
jackpooleyml
left a comment
There was a problem hiding this comment.
LGTM. Why remove WalletConnect? What was this used for, I'm guessing for adding EOA signers?
|
I cannot find what comments are left to be resolved for this merge request to close, I've addressed all the comments I've found! |
|
giving up here for now, will come back with final main net addresses |
Description
add contracts to main net
removes wallet connect from app load
Additional context
fixes auth-test by removing walletconnect