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.

Sign up for GitHub

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

Jump to bottom

[DX-2784] feat: added zkevm transaction receipt acquirement method #185

Merged
merged 4 commits into from
May 7, 2024
Merged

[DX-2784] feat: added zkevm transaction receipt acquirement method #185

merged 4 commits into from
May 7, 2024

Conversation

YermekG
Copy link
Contributor

@YermekG YermekG commented May 6, 2024 •
edited by nattb8
Loading

Summary

  • Added transaction receipt method to retrieve descriptive info on the status of zkEVM transaction via transaction hash.
  • Updated the game bridge index.html file (built from ts-immutable-sdk 56e632a)

Customer Impact

Customers can use ZkEvmGetTransactionReceipt to retrieve transaction information of a given transaction hash. The TransactionReceiptResponse.cs file contains the full response data model.

Other things to consider:

  • Sample app is updated with new SDK changes
  • Updated public documentation with new SDK changes (Immutable X and Immutable zkEVM)
  • Sample game is updated with new SDK changes
  • Replied to GitHub issues

@YermekG YermekG requested a review from a team as a code owner May 6, 2024 03:08
@YermekG YermekG changed the title [DX-2784] feat: added zkEVM transaction receipt acquirement method [DX-2784] feat: added zkevm transaction receipt acquirement method May 6, 2024
@YermekG YermekG force-pushed the feat/transfer_check branch from 53c32b5 to 6fc7fe2 Compare May 6, 2024 03:15
@YermekG YermekG requested a review from nattb8 May 6, 2024 03:15
@YermekG YermekG force-pushed the feat/transfer_check branch from 6fc7fe2 to c01fbc5 Compare May 6, 2024 03:17
@YermekG YermekG force-pushed the feat/transfer_check branch from 8ffd956 to ce28009 Compare May 6, 2024 22:01
@YermekG YermekG enabled auto-merge May 6, 2024 22:03
@YermekG YermekG merged commit f76e350 into main May 7, 2024
13 checks passed
@YermekG YermekG deleted the feat/transfer_check branch May 7, 2024 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nattb8 nattb8 nattb8 approved these changes

Assignees
No one assigned
Labels
feature
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@YermekG @nattb8