MetaMask / MetaMask/mpc-core-kit-swift
Improve Error Messages
- Dominant language
- Swift
- Stars
- 0
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
## Description
Currently, the error messages are not efficient and developer friendly. It's not easy to understand what's wrong. Below are few instances
- getDeviceFactor - When there is not device factor, the error is `The operation couldn’t be completed. (mpc_core_kit_swift.CoreKitError error 0.)`. This error doesn't make me understand as a developer what's the issue or even show meaningful message to the user.
- enableMFA - When MFA is already enabled, it throws `The operation couldn’t be completed. (mpc_core_kit_swift.CoreKitError error 24.)` instead of throwing MFA is already enabled.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by locating the getDeviceFactor and enableMFA implementations, then find the CoreKitError definitions or mapping used for their failures. Compare the current errors with the two cases described and determine the expected developer- and user-facing messages; done means those conditions produce meaningful, distinguishable errors.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- swift
- Domain
- mobile-dev
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100