Skip to content

Conversation

@VictorTrustyDev
Copy link

@VictorTrustyDev VictorTrustyDev commented Jan 1, 2024

New rules:

  • Fee must be empty (compatible with MinGasPrice is zero)
  • Or fee must be single element array of EVM-denom
  • Otherwise reject

@VictorTrustyDev VictorTrustyDev added the enhancement Enhancement / bug-fixes or re-work on old feature label Jan 1, 2024
@VictorTrustyDev VictorTrustyDev self-assigned this Jan 1, 2024
VictorEscan
VictorEscan previously approved these changes Jan 1, 2024
VictorEscan
VictorEscan previously approved these changes Jan 1, 2024
@VictorTrustyDev VictorTrustyDev force-pushed the imp/reject-cosmos-txs-that-fee-is-not-evm-denom branch from 1027579 to c8676d1 Compare January 1, 2024 20:00
@VictorTrustyDev VictorTrustyDev merged commit f0e8572 into main Jan 1, 2024
@VictorTrustyDev VictorTrustyDev deleted the imp/reject-cosmos-txs-that-fee-is-not-evm-denom branch January 1, 2024 20:00
VictorTrustyDev added a commit that referenced this pull request Jan 1, 2024
…-not-evm-denom-backport-37

imp(fee): reject cosmos txs that fee denom is not evm denom (backport #37)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement / bug-fixes or re-work on old feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants