-
Notifications
You must be signed in to change notification settings - Fork 8
docs: keep javascript tabs on sending guides #943
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?
Conversation
🌿 Documentation Preview
|
| <Tabs> | ||
| <Tab title="React"> | ||
| <Markdown src="react.mdx" /> | ||
| <Tab title="API"> |
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.
i thought were were going to have the js tab be the default?
| <Tabs> | ||
| <Tab title="React"> | ||
| <Markdown src="react.mdx" /> | ||
| <Tab title="API"> |
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.
nit: pre-existing issue
| <Tab title="API"> | |
| <Tab title="API" language="bash">> |
| <Tabs> | ||
| <Tab title="React"> | ||
| <Markdown src="react.mdx" /> | ||
| <Tab title="API"> |
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.
| <Tab title="API"> | |
| <Tab title="API" language="bash"> |
| <Tabs> | ||
| <Tab title="React"> | ||
| <Markdown src="react.mdx" /> | ||
| <Tab title="API"> |
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.
| <Tab title="API"> | |
| <Tab title="API" language="bash"> |
| <Tab title="React"> | ||
| See the [`usePrepareCalls` react | ||
| hook](/wallets/reference/account-kit/react/hooks/usePrepareCalls) | ||
| <Tab title="API"> |
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.
| <Tab title="API"> | |
| <Tab title="API" language="bash"> |
Motivation
APIexamples andJavaScriptSDK examples while removing theReacttab, and displayAPIfirst.Description
Reacttabs withAPItabs and addedJavaScripttabs infern/wallets/pages/transactions/send-transactions/index.mdx,send-batch-transactions/index.mdx,send-parallel-transactions/index.mdx, andusing-eip-7702/index.mdx.APItab linking towallet_prepareCallsand aJavaScripttab linking to the SDKprepareCallswhere applicable, and removed React-specific prepare-calls content.APItabs are listed first across the modified pages and removed duplicated API blocks.Testing
eslint --fixandprettier --write --log-level silent, both of which completed successfully.Codex Task