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

upgrade fuel-core to 0.41.0 #1575

Open
hal3e opened this issue Jan 15, 2025 · 0 comments
Open

upgrade fuel-core to 0.41.0 #1575

hal3e opened this issue Jan 15, 2025 · 0 comments
Assignees

Comments

@hal3e
Copy link
Contributor

hal3e commented Jan 15, 2025

No description provided.

@hal3e hal3e self-assigned this Jan 15, 2025
hal3e added a commit that referenced this issue Jan 17, 2025
closes:#1575
closes #1477

# Summary
`fuels-rs` is using `fuel-core-client` to communicate with the node.

`fuel-core-client`:
- added a new `TransactoinType` enum with an `Unknown` variant to
`fuel-core-client`.
- added `Unknown` variants to `ConsensusParameters` and all of its
fields. `cynic` will fallback to this variant.
  
`fuel-core` PR: FuelLabs/fuel-core#2154

Checklist:
- [x] transaction: `fuel-core-client`
- [x] add new `TransactionType` with the `Unknown` variant in
`fuel-core-client`
  - [x] refactor `fuels-rs`

- [x] coin-type: `fuels-rs`
- [x] `fuel-core-client` already has an `Unknown` variant - refactor
`fuels-rs`

- [x] consensus-parameters: `fuel-core-client`
  - [x] add `Unknown` variants to all the version enums
  - [x] refactor code

- [x] block and headers: `fuel-core-client`
  - [x] add `Unknown` variants to all the version enums
  - [x] refactor code

# Breaking Changes
- added `Unknown` variant to `TransactionType` and `CoinType`
- `CoinType` methods `id`, `owner`, `asset_id` now return `Option`
- `get_message_proof` now returns `Result<Option<MessageProof>>` instead
of `Result<MessageProof>`
- `get_balances` now returns  `Result<HashMap<String, u128>>

# Checklist

- [x] All **changes** are **covered** by **tests** (or not applicable)
- [x] All **changes** are **documented** (or not applicable)
- [x] I **reviewed** the **entire PR** myself (preferably, on GH UI)
- [x] I **described** all **Breaking Changes** (or there's none)
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

No branches or pull requests

1 participant