Skip to content

Start with Ethereum web3 #14947

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

Merged
merged 45 commits into from
Mar 25, 2025
Merged

Start with Ethereum web3 #14947

merged 45 commits into from
Mar 25, 2025

Conversation

corwintines
Copy link
Member

@corwintines corwintines commented Feb 19, 2025

@github-actions github-actions bot added dependencies 📦 Changes related to project dependencies tooling 🔧 Changes related to tooling of the project labels Feb 19, 2025
Copy link

netlify bot commented Feb 19, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 9878674
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/67dda9a599cdfd00080ae6d6
😎 Deploy Preview https://deploy-preview-14947--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 52 (🔴 down 8 from production)
Accessibility: 95 (🟢 up 1 from production)
Best Practices: 89 (🔴 down 8 from production)
SEO: 98 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@github-actions github-actions bot added content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program labels Feb 27, 2025
Copy link
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

@corwintines gj! left a few thoughts, overall looks great.

@github-actions github-actions bot added the config ⚙️ Changes to configuration files label Mar 19, 2025
Copy link
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

@corwintines thanks for the adjustments. Looks good on my side. I'm pushing two minor commits to polish the things mentioned.

height={30}
/>
</div>
<p className="text-body">{wallet.name}</p>
Copy link
Member

Choose a reason for hiding this comment

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

The LinkBox is fine but we are missing the LinkOverlay that needs to wrap the description/text for the link. Pushing a commit for this.


return (
<QueryClientProvider client={queryClient}>
<WalletProviders locale={router.query.locale as string}>
Copy link
Member

Choose a reason for hiding this comment

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

We should use now the useLocale hook to get it. Pushing a commit for this as well.

@corwintines corwintines merged commit d7752b7 into dev Mar 25, 2025
8 of 10 checks passed
@corwintines corwintines deleted the web3-integration branch March 25, 2025 17:18
This was referenced Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config ⚙️ Changes to configuration files content 🖋️ This involves copy additions or edits dependencies 📦 Changes related to project dependencies tooling 🔧 Changes related to tooling of the project translation 🌍 This is related to our Translation Program
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants