Skip to content

Fix: "value" is supported#565

Open
King31T wants to merge 2 commits intotronprotocol:masterfrom
King31T:fix_ethCall
Open

Fix: "value" is supported#565
King31T wants to merge 2 commits intotronprotocol:masterfrom
King31T:fix_ethCall

Conversation

@King31T
Copy link
Copy Markdown
Contributor

@King31T King31T commented Mar 31, 2026

No description provided.

| gas | QUANTITY | Not supported. The value is 0x0 |
| gasPrice | QUANTITY | Not supported. The value is 0x0 |
| value | QUANTITY | Not supported. The value is 0x0 |
| value | QUANTITY | The integer of value sent with this transaction encoded as hexadecimal |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amount of TRX transferred into the contract. (Unit: sun, format: Hex)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In some scenarios, such as swap router contracts, TRX is not transferred to the called contract

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about this one? or another simple description, I think we need to clarify that this refers to TRX. (Optional) Amount of TRX sent with the call (Unit: sun, Format: Hex). Maps to msg.value to simulate contract logic; no actual transfer occurs.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amount of TRX sent with the transaction (Unit: sun, Format: Hex).

Keep the “transaction” consistent with the original content of eth_call. Okay?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok.

Copy link
Copy Markdown
Contributor

@vivian1912 vivian1912 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

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

Successfully merging this pull request may close these issues.

4 participants