Skip to content

Conversation

@wodsuz
Copy link
Contributor

@wodsuz wodsuz commented Jun 8, 2022

Added testnet button on connections page
checked their rpc url and chainid to make sure they are working correctly
Changed the display of radio buttons little bit to make it look better on mobile.
image
image

url += `&confirmationTime=${confirmationTime}`;
}
url += `&network=${network.toLowerCase()}`;
url += `&network=${network}`;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not lowercase?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cause of adding Automates rinkeby testnet rpc i think,
it should be
https://[email protected]&apiKey=&draft=true&network=arbitrumRinkeby

not
https://[email protected]&apiKey=&draft=true&network=arbitrumrinkeby

with lower case it was giving a wrong rpc url.

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.

3 participants