MetaMask / MetaMask/metamask-extension

[Bug]: transaction metrics does not capture the information if transaction used advance default estimate

Open
#14,456 0 comments 0 reactions 0 assignees View on GitHub
area-gas area-metrics area-transactions
Dominant language
TypeScript
Stars
13.2k
Forks
5.6k
Avg merge
2d 5h
Merged PRs (30d)
451

Description

In transaction metrics we capture the estimate used for transaction `low / medium / high / custom / dapp-suggested`. But we do not capture information if custom (or advance) estimate used by transaction is the default estimate saved by the user.

Here we can capture metrics at 2 places:
1. as user updates advance default we can capture user setting metrics
2. is transaction created by user is using advance default can be captured in transaction metrics

cc @kevinghim , @brad-decker

Contributor guide

Open the contributing guide

Research direction

Start by locating the transaction metrics entry point and the handling for user-saved advanced default estimates. Trace both settings updates and transaction creation to determine where the estimate source is recorded. Done means metrics capture when a transaction uses the user's default advanced estimate, including the setting update event if applicable.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
analytics
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.