Skip to content
This repository has been archived by the owner on Oct 25, 2024. It is now read-only.

Script and Mint transactions are replaced to empty Script transactions #1357

Closed
a-filippo opened this issue Sep 17, 2023 · 1 comment · Fixed by #1358
Closed

Script and Mint transactions are replaced to empty Script transactions #1357

a-filippo opened this issue Sep 17, 2023 · 1 comment · Fixed by #1358
Assignees
Labels
bug Something isn't working

Comments

@a-filippo
Copy link
Contributor

Issue

Description

file: packages/fuel-indexer/src/executor.rs, fn retrieve_blocks_from_node

_ => Transaction::default(),

Script and Mint transactions are replaced to Transaction::default(). By this reason inside my indexer these types of transactions are broken. I checked the commit history and file changes, and decided that this problem did not exist before.
Anyway I have a branch which is the expected behavior in my opinion: https://github.com/FuelLabs/fuel-indexer/compare/develop...a-filippo:fuel-indexer:added-script-mint?expand=1

I am attaching an example of log, which shows that both block transactions has Script type with empty information.

2023-09-18 01 07 49

Versions

  • fuel-indexer: 0.20.7
  • forc-index: 0.20.7
@ra0x3 ra0x3 added the bug Something isn't working label Sep 18, 2023
@ra0x3
Copy link
Contributor

ra0x3 commented Sep 18, 2023

  • Doh! Thanks @a-filippo it appears we didn't go back to add some of these cc @deekerno
  • We'll try to get your commit in, so that you can get contribution
  • Another note @a-filippo, you'll have to sign something if you want to get your commit into the repo 😅 (standard operating procedure)
    • No rush here, but if not done in a few hours, we'll probably just commit your example ourselves so as to unblock you ASAP :)

@ra0x3 ra0x3 linked a pull request Sep 18, 2023 that will close this issue
7 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants