Comments (5)
Hey team! Please add your planning poker estimate with Zenhub @digiwand @segun @blackdevelopa @seaona
from metamask-extension.
Pending to clarify with product, if it really make sense to display the False Positive link, for transactions which failed to be verified - because they are not "false positives" per se.
Since it's a minor issue, we can double-check this with Barbara next week, so we get a confirmation on the expected behaviour here
cc @digiwand
from metamask-extension.
thanks @seaona! will be on standby
from metamask-extension.
My opinion is we should allow users to report failed transactions as well. I see on the video above that the result type of those show up as benign
on the form - could we have it show up as failed
? That way blockaid could triage it on their side and differentiate false positives from failed reports.
from metamask-extension.
Oh it seems the result type showing up as benign was a bug on the form side that was already fixed so ignore this part. References: #22536 & slack message
from metamask-extension.
Related Issues (20)
- UX: Mobile: Basic Functionality Component
- UX: Mobile: Basic Functionality Component
- transaction-utils determineTransactionType does not handle missing `to` parameter
- [Bug]: When using a hardware wallet,unable to establish an effective connection with DApp. HOT 2
- [Bug/UX]: Dashboard shows `USD` and confusing warning instead of balance when on private network HOT 1
- [Bug]: Unapproved transaction stuck, settings and other buttons disabled HOT 42
- [Bug]: Failed to execute 'bound' on 'IDBKeyRange': The parameter is not a valid key.\n DataError: Failed to execute 'bound' on 'IDBKeyRange': The parameter is not a valid key. HOT 2
- [development] pop up an OS notification when a development build finishes
- [Bug]: still loading forever and no password prompt after update to 11.11.4 HOT 7
- [Bug]: HOT 1
- [Bug]: Import token in @MetaMask seems to be broken for Pulsechain HOT 1
- [Bug]: Polling Infura mainnet regardless of network settings HOT 1
- `lavamoat:background:auto` mentioned in ReadMe no longer exists HOT 1
- Conflux's Logo Have Bad Display in Dark Mode
- Fix "Settings - general tab, validate the change language..." flaky tests
- Keystone - GPL package update to non-copyleft license package
- [Bug]: Metamask Expanded View Requires Password Re-entry After Browser Reopen HOT 4
- Fix "Sentry errors after initialization, after opting int..." flaky tests
- Bug: adding account does not close snap UI and open extension confirmation screens HOT 2
- Set up `@typescript-eslint/consistent-type-definitions` rule to ban `interface` keyword
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from metamask-extension.