Replace gas price with fee

[gterzian]

No description provided.

[gterzian]

@aoudiamoncef Ready for review.

Which branch did you mention I should target?

Where do I update the docs?

[aoudiamoncef]

Which branch did you mention I should target?

create a new branch from : bugfix/op-wrong-types

Where do I update the docs?

update openrpc.json file which is located on the root of massa repository.

[gterzian]

@aoudiamoncef Thanks for the info.

create a new branch from : bugfix/op-wrong-types

What do you mean? Is that the name of a branch I should branch off from(or target for merge)? I can't see it online actually, have you pushed it already?

[aoudiamoncef]

It's here

IMHO, we could wait until merging this PR in main and back to update openrpc.json file.

[gterzian]

IMHO, we could wait until merging this PR in main and back to update openrpc.json file.

@aoudiamoncef Since your PR has now merged, could you please tell me what I exactly should update?

[aoudiamoncef]

Hi @gterzian

You have to update this JSON objects to match the current MassaAPI input/output

https://github.com/massalabs/massa/blob/mai(n/massa-node/base_config/openrpc.json#L1865-L1936

May be, we should rename the type to match exactly it's name in the Massa API.

[gterzian]

@aoudiamoncef Ok I have updated the openrpc.json file in massalabs/massa#3173, should we try to merge this one after a review?

[gterzian]

@aoudiamoncef What do you think is the right approach to merge this for provisional use in massalabs/massa#3181 (via massalabs/massa#3173), while preventing breakage for the hackaton? Use a new tag like 0.9? cc @damip

[aoudiamoncef]

IMHO targeting testnet_17 is the solution to avoid breaking changes.

[damip]

@aoudiamoncef What do you think is the right approach to merge this for provisional use in massalabs/massa#3181 (via massalabs/massa#3173), while preventing breakage for the hackaton? Use a new tag like 0.9? cc @damip

Those are breaking changes. We can't deliver them for the hackathon without breaking testnet16 and all the tooling around massa. So let's target testnet 17 => maybe create a testnet 17 branch here like we did for the massa repo, and make massa's testet17 branch depend on it

[gterzian]

Ok, merging this one into a newly created testnet_17 branch...