reports page #518

Merged
tech merged 2 commits from SEND053 into main 2026-02-17 09:21:39 +00:00
Collaborator
No description provided.
protuberanets self-assigned this 2026-02-16 18:06:44 +00:00
protuberanets added 1 commit 2026-02-16 18:06:45 +00:00
protuberanets requested review from tech 2026-02-16 18:06:45 +00:00
tech requested changes 2026-02-16 20:01:32 +00:00
Dismissed
@@ -0,0 +17,4 @@
void update(PaymentsProvider provider) {
if (!identical(_payments, provider)) {
_payments?.removeListener(_onPaymentsChanged);
Owner

по идее должно работать без ручного управления подпиской. update должен вызываться при любом обновлении провайдера. Соответственно, можно просто из update делать rebuild. Так не работает?

по идее должно работать без ручного управления подпиской. update должен вызываться при любом обновлении провайдера. Соответственно, можно просто из update делать rebuild. Так не работает?
@@ -0,0 +28,4 @@
void update(PaymentsProvider provider) {
if (!identical(_payments, provider)) {
_payments?.removeListener(_onPaymentsChanged);
Owner

то же самое

то же самое
@@ -0,0 +103,4 @@
return null;
}
String _normalizePaymentState(String? raw) {
Owner

по идее это должен быть просто enum с mapper'ом.

по идее это должен быть просто enum с mapper'ом.
@@ -0,0 +111,4 @@
return trimmed;
}
double _parseAmount(String? amount) {
Owner

должен быть где-то стандартный разбиральщик Amount, типа DTO. Похоже на дублирование кода

должен быть где-то стандартный разбиральщик Amount, типа DTO. Похоже на дублирование кода
protuberanets added 1 commit 2026-02-17 08:17:26 +00:00
tech approved these changes 2026-02-17 09:21:32 +00:00
tech merged commit 2707acedae into main 2026-02-17 09:21:39 +00:00
tech deleted branch SEND053 2026-02-17 09:21:39 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tech/sendico#518