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.

Sign up for GitHub

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

Move fee to function call #55

Merged
merged 3 commits into from
Oct 20, 2023
Merged

Move fee to function call #55

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
7fe0140
refactor: move fee to function call
aoudiamoncef Oct 20, 2023
3c9536b
Revert "refactor: move fee to function call"
aoudiamoncef Oct 20, 2023
07932c7
fix: coins instead of fee
aoudiamoncef Oct 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions proto/commons/massa/model/v1/execution.proto
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,7 @@
}

// Read-only execution call
message ReadOnlyExecutionCall {

Check failure on line 378 in proto/commons/massa/model/v1/execution.proto

View workflow job for this annotation

GitHub Actions / check 

Previously present field "8" with name "fee" on message "ReadOnlyExecutionCall" was deleted.
// Maximum gas to spend in the execution.
uint64 max_gas = 1;
// Call stack to simulate, older caller first
Expand All @@ -393,10 +393,8 @@
//
// Whether to start execution from final or active state
bool is_final = 6;
// Coins transferred to the target address during the call
massa.model.v1.NativeAmount coins = 7;
// fee paid by the caller when the call is processed.
massa.model.v1.NativeAmount fee = 8;
massa.model.v1.NativeAmount fee = 7;

Check failure on line 397 in proto/commons/massa/model/v1/execution.proto

View workflow job for this annotation

GitHub Actions / check 

Field "7" with name "fee" on message "ReadOnlyExecutionCall" changed option "json_name" from "coins" to "fee".

Check failure on line 397 in proto/commons/massa/model/v1/execution.proto

View workflow job for this annotation

GitHub Actions / check 

Field "7" on message "ReadOnlyExecutionCall" changed name from "coins" to "fee".
}

/// Execute a bytecode
Expand All @@ -415,6 +413,8 @@
string target_function = 2;
// Parameter to pass to the target function
bytes parameter = 3;
// Coins transferred to the target address during the call
massa.model.v1.NativeAmount coins = 4;
}

// Structure describing the output of a read only execution
Expand Down
Loading