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

fix: transaction data #666

Merged
merged 1 commit into from
Sep 23, 2024
Merged

fix: transaction data #666

merged 1 commit into from
Sep 23, 2024

Conversation

fabiooshiro
Copy link
Contributor

Pull Request type

Please add the labels corresponding to the type of changes your PR introduces:

  • Feature
  • Bugfix
  • Refactor
  • Format
  • Documentation
  • Testing
  • Other:

Description

The transaction data is hardcoded

Related Issues

N/A

Testing Performed

I tested a manual transaction 0x10bf32bd3cdf4d6196d2999c7fef36114024fda8fd02eca010ac9d331315b63f to check the content.

Checklist

  • I have performed a self-review of my own code.
  • [] The tests pass successfully with cargo test.
  • [] The code was formatted with cargo fmt.
  • [] The code compiles with no new warnings with cargo build --release and cargo build --release --features runtime-benchmarks.
  • [] The code has no new warnings when using cargo clippy.
  • [] If this change affects documented features or needs new documentation, I have created a PR with a documentation update.

@Leouarz Leouarz requested a review from kroos47 September 20, 2024 10:26
@fabiooshiro
Copy link
Contributor Author

thx for approving!

@Leouarz Leouarz merged commit 27bac49 into availproject:main Sep 23, 2024
16 checks passed
@kroos47
Copy link
Contributor

kroos47 commented Sep 23, 2024

thx for approving!

Thanks for your contribution :)

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