Skip to content

Fix: add return button to SignTransaction component#200

Open
Meesut0 wants to merge 1 commit intoAbstract-Foundation:mainfrom
Meesut0:Meesut0-patch-1
Open

Fix: add return button to SignTransaction component#200
Meesut0 wants to merge 1 commit intoAbstract-Foundation:mainfrom
Meesut0:Meesut0-patch-1

Conversation

@Meesut0
Copy link

@Meesut0 Meesut0 commented Mar 4, 2026

Fixed the SignTransaction example in
abstract-global-wallet/agw-client/actions/signTransaction.mdx.

The component previously declared a React function but did not return any JSX, making the example incomplete and not directly usable as a copy-paste snippet.

This change adds a simple <button> element that calls signTransaction, making the example consistent with the SendTransaction example and other examples mentioned in PR: #198.

Fixed the `SignTransaction` example in
`abstract-global-wallet/agw-client/actions/signTransaction.mdx`.

The component previously declared a React function but did not return any JSX, making the example incomplete and not directly usable as a copy-paste snippet.

This change adds a simple `<button>` element that calls `signTransaction`, making the example consistent with the `SendTransaction` example and other examples mentioned in PR: Abstract-Foundation#198.
@cursor
Copy link

cursor bot commented Mar 4, 2026

PR Summary

Low Risk
Low risk documentation-only change that only affects an MDX example snippet. The main risk is minor: the updated snippet could be copied into apps, but it only adds a simple button that triggers the existing signTransaction call.

Overview
Fixes the abstract-global-wallet/agw-client/actions/signTransaction.mdx usage example by adding a returned JSX <button> wired to signTransaction, making the snippet runnable/copy-pasteable instead of defining a component that never renders anything.

Written by Cursor Bugbot for commit 2275e06. This will update automatically on new commits. Configure here.

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.

1 participant