Skip to content

fix: Add userFeeLevel as dappSuggested initially when txParams contains gas values for legacy transactions#5821

Merged
OGPoyraz merged 10 commits intomainfrom
fix/add-dapp-suggested-user-fee-level
May 20, 2025
Merged

fix: Add userFeeLevel as dappSuggested initially when txParams contains gas values for legacy transactions#5821
OGPoyraz merged 10 commits intomainfrom
fix/add-dapp-suggested-user-fee-level

Conversation

@OGPoyraz
Copy link
Member

@OGPoyraz OGPoyraz commented May 16, 2025

Explanation

This PR aims to add userFeeLevel as dappSuggested initially when txParams contains gas values also for legacy transactions.

References

Changelog

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@OGPoyraz OGPoyraz requested a review from a team as a code owner May 16, 2025 09:09
@OGPoyraz OGPoyraz requested a review from a team as a code owner May 16, 2025 09:09
@OGPoyraz OGPoyraz changed the title fix: Add userFeeLevel as dappSuggested initially when txParams contains gas values fix: Add userFeeLevel as dappSuggested initially when txParams contains gas values for legacy transactions May 16, 2025
@OGPoyraz OGPoyraz enabled auto-merge (squash) May 20, 2025 09:58
@OGPoyraz OGPoyraz merged commit c767a57 into main May 20, 2025
202 checks passed
@OGPoyraz OGPoyraz deleted the fix/add-dapp-suggested-user-fee-level branch May 20, 2025 10:04
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.

3 participants