-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
Enable Nabla for pendulum #453
base: main
Are you sure you want to change the base?
Conversation
…ontract-library # Conflicts: # src/shared/useContractWrite.ts
…um-chain/portal into refactor/replace-contract-library
…um-chain/portal into refactor/replace-contract-library
✅ Deploy Preview for rococo-souffle-a625f5 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
88822c4
to
0e693ea
Compare
@TorstenStueber do we still need this PR? do we always keep this in draft? |
We need to keep it for now, unless we add a Nabla instance selector to the Portal. @vadaynujra argued against that. However, that means that we would need to keep this PR open indefinitely. We could find a different solution, e.g., an instance selector that is only shown when adding some extra parameter to the url for example. All instances can be found anyway as they are publicly visible on chain. So there is no need to hide them with all force. |
Closed #452
Don't merge this PR. This should stay unmerged for now so that we can use the Netlify deployment for testing.