Skip to content

docs: restore effectiveGasPrice example consistency #60

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

Open
monty-sei wants to merge 2 commits into
base: main
Choose a base branch
from
Open

docs: restore effectiveGasPrice example consistency #60

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4616729
docs: restore effectiveGasPrice example consistency
monty-sei Aug 12, 2026
6c827b6
Merge branch 'main' into fix/effective-gas-price-example
alexander-sei Aug 14, 2026
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
2 changes: 1 addition & 1 deletion evm/transactions.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

Sei supports EVM transactions, allowing compatibility with Ethereum-based tools and contracts. Transactions are signed messages originating from an externally owned account (EOA) that trigger state changes on the blockchain.

## Transaction Lifecycle

Check warning on line 10 in evm/transactions.mdx

View check run for this annotation

Mintlify / Mintlify Validation (seilabs) - vale-spellcheck

evm/transactions.mdx#L10 

Use sentence case for headings: 'Transaction Lifecycle'.

| Stage | Description |
| ----------------- | -------------------------------------------------------------------------------------------------------------------- |
Expand All @@ -24,7 +24,7 @@
Unlike other EVM chains where you need to wait for multiple confirmations, Sei's consensus mechanism provides immediate transaction finality. Once a transaction is included in a block, it cannot be reversed.
</Info>

## Gas Mechanics

Check warning on line 27 in evm/transactions.mdx

View check run for this annotation

Mintlify / Mintlify Validation (seilabs) - vale-spellcheck

evm/transactions.mdx#L27 

Use sentence case for headings: 'Gas Mechanics'.

Gas is a unit of computational work in the EVM that helps prevent spam and allocate resources efficiently:

Expand All @@ -44,7 +44,7 @@
- Always estimate gas before sending transactions using `eth_estimateGas`
</Warning>

## Transaction Structure

Check warning on line 47 in evm/transactions.mdx

View check run for this annotation

Mintlify / Mintlify Validation (seilabs) - vale-spellcheck

evm/transactions.mdx#L47 

Use sentence case for headings: 'Transaction Structure'.

EVM transactions in Sei follow the Ethereum transaction format with standard properties:

Expand Down Expand Up @@ -258,7 +258,7 @@
"logs": [],
"status": "0x1",
"logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
"effectiveGasPrice": "0x77359400",
"effectiveGasPrice": "0x1234",

@seidroid seidroid Bot Aug 12, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[nit] Correct fix — this is now consistent with gasPrice/maxFeePerGas above and satisfies min(baseFee + maxPriorityFeePerGas, maxFeePerGas) for any base fee.

One editorial note for whenever the deferred cleanup happens: because the example sets maxPriorityFeePerGas equal to maxFeePerGas, the fee cap is always the binding term, so this receipt only ever illustrates the cap-hit branch. evm/evm-parity/gas-and-fees.mdx:61 says the more typical outcome is the receipt reporting baseFee + maxPriorityFeePerGas. Giving the tip a distinct, smaller value than the cap would let this example demonstrate the common case instead. Not blocking, and explicitly out of scope per the PR description.

@seidroid seidroid Bot Aug 14, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[nit] Correct — this now matches maxFeePerGas/gasPrice (0x1234) in the sibling eth_getTransactionByHash block, and since maxPriorityFeePerGas also equals 0x1234, min(baseFee + tip, maxFeePerGas) resolves to exactly 0x1234 for any base fee.

Nit, and explicitly deferred in the PR description so not blocking: because the tip already saturates the cap, this example only ever illustrates the maxFeePerGas-clamped branch, never the baseFee + maxPriorityFeePerGas branch that evm/evm-parity/gas-and-fees.mdx:61 calls out as the common case. If someone later refreshes these fixtures editorially, giving maxPriorityFeePerGas a value below maxFeePerGas would make the flow demonstrate both branches (and use more realistic magnitudes than 4,660 wei).

"type": "0x2"
}
}
Expand All @@ -266,7 +266,7 @@
</Tab>
</Tabs>

## Transaction Guidelines

Check warning on line 269 in evm/transactions.mdx

View check run for this annotation

Mintlify / Mintlify Validation (seilabs) - vale-spellcheck

evm/transactions.mdx#L269 

Use sentence case for headings: 'Transaction Guidelines'.

<Info>
**Transaction Best Practices:**
Expand All @@ -285,7 +285,7 @@
| Out of Gas | Gas limit too low for the operation | Use `eth_estimateGas` to set appropriate limit |
| Contract Execution Failed | Contract function reverted | Test with `eth_call` before sending transaction |

## Additional Resources

Check warning on line 288 in evm/transactions.mdx

View check run for this annotation

Mintlify / Mintlify Validation (seilabs) - vale-spellcheck

evm/transactions.mdx#L288 

Use sentence case for headings: 'Additional Resources'.

<CardGroup cols={3}>
<Card horizontal title="RPC Reference" icon="circle-info" href="/evm/reference">
Expand Down
Loading