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 new account page + receive flow #2011

Merged
merged 32 commits into from Mar 4, 2024
Merged

:feat new account page + receive flow #2011

merged 32 commits into from Mar 4, 2024

Conversation

svenvoskamp
Copy link
Contributor

@svenvoskamp svenvoskamp commented Feb 29, 2024

Changes

  • Add enableWalletFeatures flag to web3modal options
  • Add wui-profile-button component
  • Add wui-promo component
  • Add wui-balance component
  • Add wui-tooltip-selectcomponent
  • Add wui-banner component
  • Add wui-compatible-network component
  • Added 2 composites to split account view in 2 options
  • Added w3m-receive-view
  • Added w3m-compatible-networks-view

Test the flow on /ethers-wallet or /wagmi-wallet and use email login.

Copy link

vercel bot commented Feb 29, 2024

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

Name Status Preview Updated (UTC)
web3modal-gallery ✅ Ready (Inspect) Visit Preview Mar 4, 2024 3:59pm
web3modal-laboratory ✅ Ready (Inspect) Visit Preview Mar 4, 2024 3:59pm
web3modal-react-wagmi-ex ✅ Ready (Inspect) Visit Preview Mar 4, 2024 3:59pm
web3modal-vue-wagmi-ex ✅ Ready (Inspect) Visit Preview Mar 4, 2024 3:59pm

Copy link
Contributor

github-actions bot commented Feb 29, 2024

Fails
🚫 New composites were added, but not exported in ui/utils/JSXTypeUtil.ts

Generated by 🚫 dangerJS against 4424ea4

packages/core/src/controllers/RouterController.ts Outdated Show resolved Hide resolved
packages/core/src/controllers/RouterController.ts Outdated Show resolved Hide resolved
packages/ui/src/composites/wui-balance/index.ts Outdated Show resolved Hide resolved
Comment on lines +28 to +29
const isSmall =
this.variant === 'success' || this.variant === 'transparent' || this.variant === 'shadeSmall'
Copy link
Contributor

Choose a reason for hiding this comment

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

just wondering if it would be better to have a small prop or even having a textVariant prop directly

packages/ui/src/composites/wui-qr-code/index.ts Outdated Show resolved Hide resolved
packages/ui/src/composites/wui-tabs/styles.ts Outdated Show resolved Hide resolved
@svenvoskamp svenvoskamp merged commit eede7d2 into V4 Mar 4, 2024
12 of 13 checks passed
@svenvoskamp svenvoskamp deleted the feat/wallet-features branch March 4, 2024 16:04
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.

None yet

3 participants