Skip to content
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

#902 | in low resolutions the switch network component uses the old buttons #903

Conversation

Viterbo
Copy link
Collaborator

@Viterbo Viterbo commented Dec 11, 2024

Fixes #902

This PR is included in this other PR

Description

This PR fixes not only the low-resolution buttons to switch between networks but also refactors the whole implementation from the original component code to a utils-multichain-specific shared script that incorporates reacting and updating the network parameter. The default value is based on the NETWORK_EVM_NAME environment variable.

Test scenarios

... to be continued

@Viterbo Viterbo self-assigned this Dec 11, 2024
Copy link

netlify bot commented Dec 11, 2024

Deploy Preview for teloscan-stage ready!

Name Link
🔨 Latest commit 5e71290
🔍 Latest deploy log https://app.netlify.com/sites/teloscan-stage/deploys/6759adf8e7b1e4000864d7f5
😎 Deploy Preview https://deploy-preview-903--teloscan-stage.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 11, 2024

Deploy Preview for dev-mainnet-teloscan ready!

Name Link
🔨 Latest commit 5e71290
🔍 Latest deploy log https://app.netlify.com/sites/dev-mainnet-teloscan/deploys/6759adf825c49600088a0c13
😎 Deploy Preview https://deploy-preview-903--dev-mainnet-teloscan.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 11, 2024

Deploy Preview for testnet-teloscan ready!

Name Link
🔨 Latest commit 5e71290
🔍 Latest deploy log https://app.netlify.com/sites/testnet-teloscan/deploys/6759adf8d9bc0000086a1286
😎 Deploy Preview https://deploy-preview-903--testnet-teloscan.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Viterbo Viterbo force-pushed the 902-bug--in-low-resolutions-the-switch-network-component-uses-the-old-buttons branch from 04c9584 to f486c8f Compare December 11, 2024 15:08
@Viterbo Viterbo force-pushed the 902-bug--in-low-resolutions-the-switch-network-component-uses-the-old-buttons branch from f486c8f to 5e71290 Compare December 11, 2024 15:21
@pmjanus pmjanus closed this Dec 11, 2024
@pmjanus pmjanus reopened this Dec 11, 2024
@pmjanus pmjanus closed this Dec 11, 2024
@pmjanus pmjanus reopened this Dec 11, 2024
@pmjanus pmjanus self-requested a review December 11, 2024 21:27
@pmjanus pmjanus merged commit f61165d into dev Dec 11, 2024
15 checks passed
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