Exchange settings - #1727
Merged
Merged
Conversation
…nagement - Introduced email notifications toggle in exchange app settings with corresponding localization. - Enhanced the ExchangeBitcoinWalletsScreen to manage default wallet addresses with improved UI and feedback mechanisms. - Implemented file upload functionality in ExchangeFileUploadScreen, including error handling and user feedback. - Added statistics screen for exchange with relevant data display and localization support. - Updated various models and state management to accommodate new features.
- Updated TransactionsCubit to accept an exchangeOnly parameter for filtering transactions. - Modified TransactionsState to include exchangeOnly state and adjust available filters accordingly. - Adjusted the TransactionsScreen to initialize TransactionsCubit with the new parameter. - Updated TxsFilterRow to utilize the available filters based on the exchangeOnly state.
- Added functionality to navigate to the PayRoute with the selected recipient when a recipient is chosen. - Introduced RecipientViewModel to facilitate passing recipient data during navigation.
- Added recipientType parameter to updateMyRecipient method for better API integration. - Updated deleteDefaultWallet method to include walletType and address parameters. - Adjusted related use cases and cubit to accommodate new parameters for improved wallet management.
- Enhanced the KYC document upload functionality to use base64 encoding for file data. - Updated the API request structure to include file type and size information. - Introduced new models for managing KYC document statuses and integrated them into the user summary. - Modified the file upload UI to support single file uploads and reflect the current upload status. - Added localization for success and status messages related to KYC document uploads.
kumulynja
requested changes
Jan 5, 2026
kumulynja
left a comment
Contributor
There was a problem hiding this comment.
Quickly saw these two things already, should review the rest still too.
- Changed the KYC document upload method to utilize multipart form data instead of base64 encoding. - Updated the API request structure to remove file type and size parameters, focusing on file upload via FormData. - Enhanced error handling to check for various error indicators in the API response.
- Added KYC document status handling in user summary mapping. - Updated file upload state to include user data loading status. - Improved UI interactions based on KYC document upload status, allowing re-upload for rejected files. - Localized messages for KYC upload statuses, including a new message for rejected uploads.
Contributor
Author
|
this is ready to be merged and for testing |
…and withdraw wrap it and pass their own hook, errors and loading indication to it
…ents Recipients reuse improvements
Contributor
Author
|
@kumulynja is this pr ready to go after your changes? |
8 tasks
Contributor
Yeah, I focused the review and commits mostly on the recipients stuff and that should be good now. The rest is new and should be good for now too 👍 |
i5hi
approved these changes
Jan 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
1. New Exchange Settings Menu Items
The Exchange Settings screen now includes several new functional menu entries (previously commented out/placeholder):
RecipientsScreenandRecipientsBloc2. Default Bitcoin Wallets Screen
File:
lib/features/settings/ui/screens/exchange/bitcoin_wallets_screen.dartComplete redesign with functional wallet management:
3. App Settings Screen
File:
lib/features/settings/ui/screens/exchange/app_settings_screen.dartEnhanced with new functionality:
initStateTextAlign.center,SnackBarBehavior.floating)4. Secure File Upload Screen
File:
lib/features/settings/ui/screens/exchange/file_upload_screen.dartCompletely redesigned with functional upload capability:
Upload(ready state with upload button)In Review(warning-styled badge)Accepted(success-styled badge)5. New Statistics Screen
File:
lib/features/settings/ui/screens/exchange/statistics_screen.dartBrand new screen showing user trading statistics:
6. Transactions Screen Enhancements
Files:
lib/features/settings/ui/screens/exchange/transactions_screen.dartlib/features/transactions/presentation/blocs/transactions_cubit.dartlib/features/transactions/presentation/blocs/transactions_state.dartlib/features/transactions/ui/widgets/txs_filter_row.dartThe exchange transactions screen now:
TransactionsScreenwith anexchangeOnlymode7. Recipients Screen
File:
lib/features/settings/ui/screens/exchange/recipients_screen.dartNow fully functional using the existing recipients infrastructure:
RecipientsScreenwidgetRecipientsBlocfor data management8. New Localization Strings
File:
localization/app_en.arb~40+ new localization keys added for:
retrybutton labelNavigation Flow