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

Added sending unsponsored transaction to kit demo #186

Merged

Conversation

andrewlee348
Copy link
Contributor

No description provided.

SamueleA
SamueleA previously approved these changes Nov 1, 2024
Copy link
Contributor

@SamueleA SamueleA left a comment

Choose a reason for hiding this comment

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

LGTM!

@tolgahan-arikan
Copy link
Contributor

tolgahan-arikan commented Nov 1, 2024

let's also update Polygon example to remove current active one (it doesn't actually work since contract is not deployed on Polygon) and use a native token transfer.

I'll put this in draft for now.

@andrewlee348
Copy link
Contributor Author

I noticed that the next demo didnt have sponsored transaction testing so I added that while I was at it.

@andrewlee348 andrewlee348 requested a review from SamueleA November 1, 2024 17:27
examples/next/src/app/components/Connected.tsx Outdated Show resolved Hide resolved
examples/next/src/app/components/Connected.tsx Outdated Show resolved Hide resolved
examples/react/src/components/Connected.tsx Outdated Show resolved Hide resolved
@andrewlee348 andrewlee348 marked this pull request as ready for review November 6, 2024 20:52
@SamueleA SamueleA force-pushed the 3427-add-unsponsored-transaction-option-to-kit-demo branch from 6a8eff5 to b140c59 Compare November 7, 2024 14:28
@andrewlee348 andrewlee348 requested a review from SamueleA November 7, 2024 15:28
@andrewlee348 andrewlee348 merged commit 2ef4f33 into master Nov 12, 2024
1 check passed
@andrewlee348 andrewlee348 deleted the 3427-add-unsponsored-transaction-option-to-kit-demo branch November 12, 2024 18:33
github-actions bot pushed a commit that referenced this pull request Nov 12, 2024
)

* Added sending unsponsored transaction to kit demo

* Added config sponsor list, and adjusted sponsored send transaction logic

* Changed sponsored contract address config name, used  for sendTransaction

* Moved  type to config instead of sendTransaction

* removed some comments

* Added testNetChains to config and changed logic for sendTransaction

* Update testnet check

* Update next demo, update runSendTransaction

* Fix state update on chain change, show mint button on related chains

* Added setLastTxnDataHash3 reset after chain switch

* Fixed next Connected.tsx inconsistency with react

---------

Co-authored-by: Tolgahan Arikan <tolgahan.arikan@gmail.com>
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.

3 participants