tech/sendico
2
0
Fork 0
Code Issues 30 Pull Requests Actions Packages Projects Releases Wiki Activity

refactoring for recipient addition page #344

Merged
tech merged 2 commits from SEND039 into main 2026-01-29 22:44:37 +00:00
Conversation 1 Commits 2 Files Changed 48 +1467 -536
protuberanets commented 2026-01-29 16:22:56 +00:00
Collaborator
Copy Link
No description provided.
protuberanets self-assigned this 2026-01-29 16:22:56 +00:00
protuberanets added 1 commit 2026-01-29 16:22:57 +00:00
refactoring for recipient addition page efa69b43b2
protuberanets requested review from tech 2026-01-29 16:22:57 +00:00
protuberanets changed title from refactoring for recipient addition page to WIP: refactoring for recipient addition page 2026-01-29 16:23:17 +00:00
protuberanets added 1 commit 2026-01-29 16:55:06 +00:00
hotfix for payment page to see more then one payment type a3908cebba
protuberanets changed title from WIP: refactoring for recipient addition page to refactoring for recipient addition page 2026-01-29 16:55:15 +00:00
tech approved these changes 2026-01-29 22:38:31 +00:00
frontend/pshared/lib/provider/payment/flow.dart
@@ -15,6 +15,7 @@ class PaymentFlowProvider extends ChangeNotifier {
PaymentMethodData? _manualPaymentData;
List<PaymentMethod> _recipientMethods = [];
Recipient? _recipient;
String? _selectedMethodId;
tech commented 2026-01-29 22:33:00 +00:00
Owner
Copy Link

Сейчас можно не править, но для себя понял, что связанные с UI вещи лучше укладывать в аналоги провайдера для UI - контроллеры. Так проще код получается, а провайдер фокусируется строго на одной задаче - получения и управления данными, тогда как контроллер работает как надстройка над провайдером, обеспечивая прокладку между UI и провайдером, сохраняя в себе состояния UI.

Сейчас можно не править, но для себя понял, что связанные с UI вещи лучше укладывать в аналоги провайдера для UI - контроллеры. Так проще код получается, а провайдер фокусируется строго на одной задаче - получения и управления данными, тогда как контроллер работает как надстройка над провайдером, обеспечивая прокладку между UI и провайдером, сохраняя в себе состояния UI.
frontend/pweb/lib/providers/address_book_recipient_form.dart
@@ -0,0 +13,4 @@
import 'package:pweb/models/seed_state.dart';
class AddressBookRecipientFormProvider extends ChangeNotifier {
tech commented 2026-01-29 22:35:58 +00:00
Owner
Copy Link

больше на контроллер похоже, чем на провайдер.

больше на контроллер похоже, чем на провайдер.
tech merged commit 51f5b0804a into main 2026-01-29 22:44:37 +00:00
tech deleted branch SEND039 2026-01-29 22:44:38 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
tech
Labels
Clear labels
backend
bug
Something is not working
 cicd
Continious Integration / Continuous Deployment
 docs
duplicate
This issue or pull request already exists
 enhancement
New feature
 frontend
help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 refactoring
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
protuberanets tech
No Assignees protuberanets
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tech/sendico#344
Delete Branch "SEND039"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?