feat: error code

[jaymengxy]

Related Issue

Closes #764

Summary of Changes

Need Regression Testing

• Yes
• No

Risk Assessment

• Low
• Medium
• High

Additional Notes

Screenshots (if applicable)

[github-actions]

⏳ Analyzing changes in this PR... ⏳

This might take a few minutes, please wait

📥 Commits

Analyzing changes from base (b3d04e4) to latest commit (d789e05):

• d789e05: feat: error code

📁 Files being considered (21)

🔄 app/src/main/java/com/flowfoundation/wallet/cache/AccountCacheManager.kt (2 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/cache/UserPrefixCacheManager.kt (2 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/account/AccountManager.kt (6 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/account/WalletFetcher.kt (2 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/backup/GoogleDriveBackupUtils.kt (3 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/cadence/CadenceApiManager.kt (3 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/drive/GoogleDriveAuthActivity.kt (2 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/dropbox/DropboxBackupUtils.kt (3 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/evm/EVMWalletManager.kt (5 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/flowjvm/CadenceExecutor.kt (2 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/manager/staking/StakingManager.kt (6 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/network/UserRegisterUtils.kt (3 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/page/backup/multibackup/viewmodel/BackupDropboxViewModel.kt (3 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/page/backup/multibackup/viewmodel/BackupGoogleDriveViewModel.kt (4 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/page/backup/multibackup/viewmodel/BackupRecoveryPhraseViewModel.kt (3 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/page/backup/viewmodel/BackupSeedPhraseViewModel.kt (3 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/page/restore/keystore/viewmodel/KeyStoreRestoreViewModel.kt (8 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/page/restore/multirestore/viewmodel/MultiRestoreViewModel.kt (4 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/page/wallet/confirm/presenter/WalletConfirmPresenter.kt (2 hunks)
🔄 app/src/main/java/com/flowfoundation/wallet/utils/error/Error.kt (1 hunk)
🔄 app/src/main/java/com/flowfoundation/wallet/utils/error/ErrorReporter.kt (2 hunks)

---

autogenerated by presubmit.ai

[github-actions]

✅ LGTM!

Review Summary

Commits Considered (1)

• b3d04e4: feat: error code

Files Processed (22)

• app/src/main/java/com/flowfoundation/wallet/cache/AccountCacheManager.kt (2 hunks)
• app/src/main/java/com/flowfoundation/wallet/cache/UserPrefixCacheManager.kt (2 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/account/AccountManager.kt (6 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/account/WalletFetcher.kt (2 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/backup/GoogleDriveBackupUtils.kt (3 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/cadence/CadenceApiManager.kt (3 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/drive/GoogleDriveAuthActivity.kt (2 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/dropbox/DropboxBackupUtils.kt (3 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/evm/EVMWalletManager.kt (5 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/flowjvm/CadenceExecutor.kt (2 hunks)
• app/src/main/java/com/flowfoundation/wallet/manager/staking/StakingManager.kt (6 hunks)
• app/src/main/java/com/flowfoundation/wallet/network/UserRegisterUtils.kt (3 hunks)
• app/src/main/java/com/flowfoundation/wallet/page/backup/multibackup/viewmodel/BackupDropboxViewModel.kt (3 hunks)
• app/src/main/java/com/flowfoundation/wallet/page/backup/multibackup/viewmodel/BackupGoogleDriveViewModel.kt (4 hunks)
• app/src/main/java/com/flowfoundation/wallet/page/backup/multibackup/viewmodel/BackupRecoveryPhraseViewModel.kt (3 hunks)
• app/src/main/java/com/flowfoundation/wallet/page/backup/viewmodel/BackupSeedPhraseViewModel.kt (3 hunks)
• app/src/main/java/com/flowfoundation/wallet/page/nft/nftlist/utils/NftListRequester.kt (1 hunk)
• app/src/main/java/com/flowfoundation/wallet/page/restore/keystore/viewmodel/KeyStoreRestoreViewModel.kt (8 hunks)
• app/src/main/java/com/flowfoundation/wallet/page/restore/multirestore/viewmodel/MultiRestoreViewModel.kt (4 hunks)
• app/src/main/java/com/flowfoundation/wallet/page/wallet/confirm/presenter/WalletConfirmPresenter.kt (2 hunks)
• app/src/main/java/com/flowfoundation/wallet/utils/error/Error.kt (1 hunk)
• app/src/main/java/com/flowfoundation/wallet/utils/error/ErrorReporter.kt (2 hunks)

Actionable Comments (0)

Skipped Comments (3)

• app/src/main/java/com/flowfoundation/wallet/manager/evm/EVMWalletManager.kt [81-82]

  best practice: "Private function exposed through public property"

• app/src/main/java/com/flowfoundation/wallet/manager/flowjvm/CadenceExecutor.kt [770-772]

  enhancement: "Improve error handling by storing exception before logging"

• app/src/main/java/com/flowfoundation/wallet/page/restore/keystore/viewmodel/KeyStoreRestoreViewModel.kt [238-243]

  best practice: "Consider adding error details to error reporting"