Skip to content

chore: remove keys-dev.coinbase.com from playground - #407

Open
fan-zhang-sv wants to merge 1 commit into
masterfrom
toshi/remove-dev-keys-domain
Open

fan-zhang-sv wants to merge 1 commit into
masterfrom
toshi/remove-dev-keys-domain

Conversation

@fan-zhang-sv

Copy link
Copy Markdown
Collaborator

Summary

Remove https://keys-dev.coinbase.com/connect from the playground SCW URL picker. Prod, beta, and localhost remain.

How did you test your changes?

  • Updated EIP1193ProviderContextProvider tests to use keys-beta.coinbase.com
  • Ran ConfigContextProvider tests (5 passed)
  • Provider-context suite needs a package build; CI will cover it

Generated with Toshi

Drop the keys-dev environment from the playground SCW URL picker.
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

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.

2 participants