-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
style pass over features #2602
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
base: main
Are you sure you want to change the base?
style pass over features #2602
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| ## Next steps | ||
|
|
||
| You can contact our [Business Team](https://calendly.com/elishamm/sales) for more information on how to access this feature. | ||
| - Contact our [Business Team](https://calendly.com/elishamm/sales) for more information on how to access this feature. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yashovardhan this is a very brittle way of offering support. Is it possible to use or create a generic bd contact that isnt tied to one individual?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
at this point no, we have only one BD person who is handling things. Even our website links to the same. Once that is resolved, we can change things here.
yashovardhan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good. Left a comment for the BD support call
| ## Next steps | ||
|
|
||
| You can contact our [Business Team](https://calendly.com/elishamm/sales) for more information on how to access this feature. | ||
| - Contact our [Business Team](https://calendly.com/elishamm/sales) for more information on how to access this feature. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
at this point no, we have only one BD person who is handling things. Even our website links to the same. Once that is resolved, we can change things here.
Description
Light style and proof pass:
Removes latin eg/ie
Simplifies language instances
Aligns with style such as uses Next Steps in favor of multiple notes
Where possible, migrates from Web3 Auth to Embedded Wallets
Fixes links in components
Issue(s) fixed
Partial fix for 2483
Preview
Checklist
External contributor checklist
Note
Broad documentation style pass across Embedded Wallets features and related components.
## Next steps) and simplifies phrasing across multiple.mdxpages (funding, MPC, NFT minting, server-side verification, session management, smart accounts, wallet pregeneration, external wallets, whitelabel)Web3AuthwithEmbedded Walletswhere appropriate; updates links to authentication docs and dashboardPregenerateWalletandPregenrateSmartAccount: revise parameter descriptions, fix links, mark optional fields, minor accessibility/ARIA label improvements; no logic changesWritten by Cursor Bugbot for commit 9505a26. This will update automatically on new commits. Configure here.