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

fix: upgrade fuels-code-gen to 0.49 #1488

Merged
merged 1 commit into from
Dec 1, 2023
Merged

fix: upgrade fuels-code-gen to 0.49 #1488

merged 1 commit into from
Dec 1, 2023

Conversation

ra0x3
Copy link
Contributor

@ra0x3 ra0x3 commented Nov 30, 2023 •
edited
Loading

Thanks for opening a PR with the Fuel Indexer project. Please review the Checklist below and ensure you've completed all of the necessary steps to make this PR review as painless as possible.

Checklist

  • Ensure your top-level commit message is in line with our contributor guidelines.
  • Please add proper labels.
  • If there is an issue associated with this PR, please link the issue (right-hand sidebar)
  • If there is not an issue associated with this PR, add this PR to the "Fuel Indexer" project (right-hand sidebar)
  • Please allow Codeowners at least 24 hours to do a first-pass review.
  • Please add thoroughly detailed testing steps below.
  • Please keep your Changelog message short and sweet.

Description

Testing steps

  • Since configurables were added to test indexer, it should compile with no errors

Changelog

  • fix: upgrade fuels-code-gen to 0.49

@ra0x3 ra0x3 mentioned this pull request Nov 30, 2023
Closed
@ra0x3 ra0x3 force-pushed the rashad/1484-asset-id-name-collission branch from 375baa5 to fdb9ecc Compare November 30, 2023 17:22
@ra0x3 ra0x3 changed the title spike fix: upgrade fuels-code-gen to 0.49 Nov 30, 2023
@ra0x3 ra0x3 self-assigned this Nov 30, 2023
@ra0x3 ra0x3 added the bug Something isn't working label Nov 30, 2023
@ra0x3 ra0x3 linked an issue Nov 30, 2023 that may be closed by this pull request
Errors "expected AssetId, found [u8; 32] arguments to this function are incorrect" and "Unrecognized primitive type: str." during forc index build #1484
Closed
@ra0x3 ra0x3 force-pushed the rashad/1484-asset-id-name-collission branch from fdb9ecc to d5db97a Compare November 30, 2023 17:52
@ra0x3 ra0x3 force-pushed the rashad/1484-asset-id-name-collission branch from d5db97a to b12ea0d Compare November 30, 2023 18:32
@ra0x3 ra0x3 marked this pull request as ready for review November 30, 2023 20:16
@ra0x3 ra0x3 merged commit 008a7e0 into develop Dec 1, 2023
@ra0x3 ra0x3 deleted the rashad/1484-asset-id-name-collission branch December 1, 2023 12:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lostman lostman lostman approved these changes

@deekerno deekerno deekerno approved these changes

Assignees

@ra0x3 ra0x3

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
3 participants
@ra0x3 @lostman @deekerno