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

feat: keep networks state in query params #1307

Conversation

chrstph-dvx
Copy link
Contributor

Summary

See #1295

Steps to test

@chrstph-dvx chrstph-dvx self-assigned this Nov 22, 2023
@cla-bot cla-bot bot added the cla-signed label Nov 22, 2023
Copy link

vercel bot commented Nov 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
arbitrum-token-bridge ✅ Ready (Inspect) Visit Preview Jan 16, 2024 2:59pm

@chrstph-dvx chrstph-dvx marked this pull request as draft November 22, 2023 18:20
@chrstph-dvx chrstph-dvx changed the title Replace useNetworksAndSigners usage with useNetworks/useNetworksRelationship feat: Replace useNetworksAndSigners usage with useNetworks/useNetworksRelationship Nov 22, 2023
@chrstph-dvx chrstph-dvx changed the title feat: Replace useNetworksAndSigners usage with useNetworks/useNetworksRelationship feat: Replace useNetworksAndSigners usage with new hooks Nov 28, 2023
@@ -669,6 +642,12 @@ export function TransferPanelMain({
const maxAmount = nativeCurrencyBalanceFloat - estimatedTotalGasFees * 1.4
// make sure it's always a positive number
// if it's negative, set it to user's balance to show insufficient for gas error
console.log(
Copy link
Member

Choose a reason for hiding this comment

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

nit: remove console.log

})
if (!didUpdateUrlWithSanitizedValues) {
// On the first render, update query params with the sanitized values
console.log('initial set query params in useNetworks')
Copy link
Member

Choose a reason for hiding this comment

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

nit: console.log

@spsjvc spsjvc changed the title feat: Replace useNetworksAndSigners usage with new hooks feat: keep networks state in query params Jan 16, 2024
spsjvc
spsjvc previously approved these changes Jan 16, 2024
Copy link
Member

@spsjvc spsjvc left a comment

Choose a reason for hiding this comment

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

L F G

@spsjvc spsjvc merged commit e60cf7e into master Jan 16, 2024
23 checks passed
@spsjvc spsjvc deleted the 1298-replace-usenetworksandsigners-with-usenetworks-and-usenetworksrelationship branch January 16, 2024 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants