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

Виджет страницы занимается не отображением, а обработкой данных #155

New Issue
Closed
opened 2025-12-24 19:26:30 +00:00 by tech · 0 comments
tech commented 2025-12-24 19:26:30 +00:00
Owner
Copy Link

frontend/pweb/lib/pages/payment_methods/page.dart:
из-за того, что данные готовятся и обрабатываются в виджете, есть какой-то странный метод sync с провайдеров PaymentFlow. В результате, когда в котировке надо достать данные текущего платежного метода, приходится какую-то странную конструкцию из двух провайдеров обрабатывать. Как правильно:

  • обработкой данных (сортировка, фильтрация, преобразование) должны заниматься только провайдеры
  • виджеты данные только отображают (то есть берут из провайдера подготовленные конкретно для них данные, ничего к ним не добавляя и не отнимая) и отдают провайдерам команды на их изменение / доставку / обновление
  • не очень ясно, что такое PaymentFlowProvider. Кажется, он должен быть надстройкой над PaymentMethods и помогать потребителям понять, какой именно сейчас платежный метод выбран в качестве цели перевода

Итого, важный принцип: виджеты данные показывают, используя провайдеры как источники собственно данных.

frontend/pweb/lib/pages/payment_methods/page.dart: из-за того, что данные готовятся и обрабатываются в виджете, есть какой-то странный метод sync с провайдеров PaymentFlow. В результате, когда в котировке надо достать данные текущего платежного метода, приходится какую-то странную конструкцию из двух провайдеров обрабатывать. Как правильно: - обработкой данных (сортировка, фильтрация, преобразование) должны заниматься только провайдеры - виджеты данные только отображают (то есть берут из провайдера подготовленные конкретно для них данные, ничего к ним не добавляя и не отнимая) и отдают провайдерам команды на их изменение / доставку / обновление - не очень ясно, что такое PaymentFlowProvider. Кажется, он должен быть надстройкой над PaymentMethods и помогать потребителям понять, какой именно сейчас платежный метод выбран в качестве цели перевода Итого, важный принцип: виджеты данные показывают, используя провайдеры как источники собственно данных.
tech added the bugfrontend labels 2025-12-24 19:26:30 +00:00
protuberanets was assigned by tech 2025-12-24 19:26:30 +00:00
protuberanets added reference SEND018 2025-12-26 12:16:09 +00:00
protuberanets changed reference from SEND018 to SEND020 2025-12-26 17:40:15 +00:00
Sign in to join this conversation.
SEND020
Branches Tags
Labels
Clear labels
backend
bug
Something is not working
 cicd
Continious Integration / Continuous Deployment
 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
 wontfix
This won't be fixed
No Label bug frontend
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
protuberanets tech
No Assignees protuberanets
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tech/sendico#155
Delete Branch "%!s()"

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?