Add latest block time and spendable resources with exclusion #833

MujkicA · 2023-02-10T15:22:51Z

Closes #832

This PR changes provider.get_spendable_resources to accept a filter which let's you define the target asset, amount, owner and also specify excluded resource ids.
It also adds provider.get_inputs_for_filter which works similarly to wallet.get_inputs_for_amount but with the advantage of being able to specify excluded ids.

It also adds a convenience method to the provider to retrieve the latest block time.

let filter = ResourceFilter {
            from: wallet.address().clone(),
            amount: coin_amount_1,
            excluded_utxos: vec![coin_2_utxo_id],
            excluded_message_ids: vec![message_id],
            ..Default::default()
        };
let resources = provider.get_spendable_resources(filter).await.unwrap();

While implementing the above, I made a bug which wasn't caught by our tests.
Because of that, I also added a test for wallet.transfer() for transferring asset ids other than the base asset.

- Add README FAQ instructions for how to run the docs locally - Hide blank debugging pages from nav - Reorganize navigation so the "Getting Started" section is not bloated - Add a page for testing basics --------- Co-authored-by: Halil Beglerović <git@hal3e.io> Co-authored-by: Ahmed Mujkic <32431923+MujkicA@users.noreply.github.com>

Enable the users to add a custom asset to contract calls by providing an asset_id, amount, and an optional address for the recipient of the generated outputs. closes #803

Co-authored-by: Ahmed Sagdati <ahmed.sagdati.ets@gmail.com>

Co-authored-by: Ahmed Mujkic <ahmedmujkic2@gmail.com>

Update `fuels-rs` to the latest `fuel-core 0.16.1` and latest `sway 0.34.0`. Due to changes in `fuel-asm` for GTF, this is causing yet another circular upgrade issue. - Used `{ workspace = true }` syntax to use one version across the project and simplify integration with new changes in the future. - Added `ci_checks.sh` script to compile all tests to simplify integration of changes like this in the future. - Update contracts to be compatible with `sway 0.34.0`. - Updated tests to be compatible with `fuel-core 0.16.1`. related PR: FuelLabs/sway#3961 --------- Co-authored-by: green <xgreenx9999@gmail.com>

This fixes the current CI issue we have.

###### Description of changes Consolidate common package metadata fields into the workspace root (e.g. version, license, authors)

###### Description of changes Adds a step to dry run the publish crates action to help identify potential issues before publishing Also removes usage of the deprecated rust-action Simplifies tag version validation

###### Description of changes Utilize workspace package to bump most crate versions in a single file

…819) Abigen moved into a new crate named `fuels-code-gen` so that the indexer team might use it.

add error handling section for the Call response page in the docs

Closes #742 Forwarding CallParameters with an amount > 0 to a method that is not annotated as payable results in a AssetsForwardedToNonPayableMethod error. BREAKING CHANGE: `.call_params()` now returns a Result --------- Co-authored-by: Halil Beglerović <git@hal3e.io> Co-authored-by: Ahmed Sagdati <ahmed.sagdati.ets@gmail.com>

….26. (#827) This updates all crates for the `fuel-asm` refactor introduced here: FuelLabs/fuel-vm#283. You can learn more about the change at this PR. The aim is to propagate this last API-breaking update through fuels-rs and sway before beta-3. The associated fuel-core PR can be found here: FuelLabs/fuel-core#973. Once the refactor lands upstream and the [patch] table is removed in this PR, this unblocks the associated sway update PR here: FuelLabs/sway#4004. --------- Co-authored-by: Brandon Kite <brandonkite92@gmail.com>

###### Description of changes Release breaking change for new fuel-asm library

Abigen with `no_std` should now be wasm friendly and use the fuels crates directly instead of through `fuels` which causes non-wasm compatible dependencies to be included atm.

iqdecay

Left some questions, looking good!

docs/src/providers/querying.md

packages/fuels-signers/src/provider.rs

packages/fuels-signers/src/wallet.rs

packages/fuels/tests/providers.rs

hal3e

Review with @segfault-magnet. Looks good. Left some nits

packages/fuels-signers/src/lib.rs

packages/fuels-signers/src/provider.rs

packages/fuels-signers/src/wallet.rs

Co-authored-by: Halil Beglerović <git@hal3e.io>

outdated

packages/fuels-signers/src/lib.rs

MujkicA and others added 27 commits January 26, 2023 21:48

add get spendable with exclusion

5bdd6de

add extract ids method

60c43a1

get exl ids

edb0093

improve extract base asset ids

d9d90c8

feat: add custom assets to contract call (#778)

74262c6

Enable the users to add a custom asset to contract calls by providing an asset_id, amount, and an optional address for the recipient of the generated outputs. closes #803

cargo doc warnings to become errors in our CI (#798)

b42f76d

feat!: deriveable Parameterize and Tokenizable (#804)

3e2a594

Co-authored-by: Ahmed Sagdati <ahmed.sagdati.ets@gmail.com>

refactor: fuels-types Error (#796)

9a2207a

fix: sign transaction in ScriptCallHandler (#806)

c9577de

Co-authored-by: Ahmed Mujkic <ahmedmujkic2@gmail.com>

Update Rust version to 1.67.0 (#814)

e9c0589

This fixes the current CI issue we have.

Use workspace level package for common crate metadata values (#816)

66598da

###### Description of changes Consolidate common package metadata fields into the workspace root (e.g. version, license, authors)

Pre-publish verification (#818)

2fccada

###### Description of changes Adds a step to dry run the publish crates action to help identify potential issues before publishing Also removes usage of the deprecated rust-action Simplifies tag version validation

Bump SDK to v0.35.0 (#817)

d7affa9

###### Description of changes Utilize workspace package to bump most crate versions in a single file

feat: Reintroduce Abigen publicly in a new crate fuels-code-gen (#…

f6e4ebd

…819) Abigen moved into a new crate named `fuels-code-gen` so that the indexer team might use it.

Bump versions to 0.35.1 (#820)

85c215d

docs: add call response error handling section (#824)

f4143f9

add error handling section for the Call response page in the docs

Fuel asm release 0.36.0 (#828)

226af64

###### Description of changes Release breaking change for new fuel-asm library

feat: add support for assert_eq logs (#811)

1eae779

ci: update forc version in CI (#830)

59235ae

feat: support all revert signals (#831)

087dfee

fix wasm compatibility when no_std is used in Abigen (#825)

1267bd9

Abigen with `no_std` should now be wasm friendly and use the fuels crates directly instead of through `fuels` which causes non-wasm compatible dependencies to be included atm.

cleanup after merge

f7796b4

add latest block time and test

4e9bf4b

MujkicA requested a review from a team February 10, 2023 15:22

MujkicA requested review from digorithm and iqdecay as code owners February 10, 2023 15:22

fix anchor

d88648b

MujkicA requested review from hal3e and segfault-magnet February 14, 2023 19:21

MujkicA added 2 commits February 14, 2023 20:22

fix path

f3dee80

improve wording in docs

99f54f2

iqdecay previously requested changes Feb 15, 2023

View reviewed changes

MujkicA and others added 4 commits February 15, 2023 13:57

move get inputs to provider

988c540

fix transfer

b0d1650

add and improve tests

2a7dd05

Merge branch 'master' into add-latest-block-time-and-exclusion

d929046

MujkicA requested a review from iqdecay February 16, 2023 11:07

MujkicA and others added 3 commits February 16, 2023 18:15

Merge branch 'master' into add-latest-block-time-and-exclusion

6daf214

Merge branch 'master' into add-latest-block-time-and-exclusion

dfc0221

fix imports

d755bb7

hal3e reviewed Feb 20, 2023

View reviewed changes

packages/fuels-signers/src/lib.rs Show resolved Hide resolved

packages/fuels-signers/src/lib.rs Outdated Show resolved Hide resolved

packages/fuels-signers/src/provider.rs Outdated Show resolved Hide resolved

packages/fuels-signers/src/wallet.rs Outdated Show resolved Hide resolved

MujkicA and others added 3 commits February 22, 2023 09:31

Update packages/fuels-signers/src/wallet.rs

75b5d54

Co-authored-by: Halil Beglerović <git@hal3e.io>

rev comments

37470e1

Merge branch 'master' into add-latest-block-time-and-exclusion

dc47377

MujkicA requested a review from hal3e February 22, 2023 09:44

MujkicA and others added 3 commits February 23, 2023 14:46

Merge branch 'master' into add-latest-block-time-and-exclusion

2f1c28a

rename get inputs via provider

aac2d0b

Merge branch 'master' into add-latest-block-time-and-exclusion

3b7b43e

segfault-magnet approved these changes Feb 27, 2023

View reviewed changes

Merge branch 'master' into add-latest-block-time-and-exclusion

4927674

digorithm reviewed Feb 27, 2023

View reviewed changes

packages/fuels-signers/src/lib.rs Show resolved Hide resolved

MujkicA requested a review from digorithm February 28, 2023 22:03

Merge branch 'master' into add-latest-block-time-and-exclusion

3cdf1a9

MujkicA merged commit 9a66942 into master Mar 3, 2023

MujkicA deleted the add-latest-block-time-and-exclusion branch March 3, 2023 15:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add latest block time and spendable resources with exclusion #833

Add latest block time and spendable resources with exclusion #833

MujkicA commented Feb 10, 2023 •

edited

Loading

iqdecay left a comment

hal3e left a comment

Add latest block time and spendable resources with exclusion #833

Add latest block time and spendable resources with exclusion #833

Conversation

MujkicA commented Feb 10, 2023 • edited Loading

iqdecay left a comment

Choose a reason for hiding this comment

hal3e left a comment

Choose a reason for hiding this comment

MujkicA commented Feb 10, 2023 •

edited

Loading