multiple payout page and small fixes #464

Merged
tech merged 2 commits from SEND047 into main 2026-02-11 15:29:28 +00:00
Collaborator
No description provided.
protuberanets self-assigned this 2026-02-10 23:49:27 +00:00
protuberanets added 1 commit 2026-02-10 23:49:27 +00:00
protuberanets requested review from tech 2026-02-10 23:49:27 +00:00
tech requested changes 2026-02-11 10:56:54 +00:00
Dismissed
@@ -11,3 +11,3 @@
final AccountResponse account;
final TokenData pendingToken;
final String destination;
@JsonKey(name: 'destination')
Owner

Поправь, пожалуйста, здесь имя переменной и атавизм в бэке на Target тоже, чтобы не расходилось.

Поправь, пожалуйста, здесь имя переменной и атавизм в бэке на Target тоже, чтобы не расходилось.
@@ -19,2 +16,2 @@
bool get canResendVerification =>
errorCode == 400 || errorCode == 410 || errorCode == 500;
ErrorResponse? get errorResponse =>
_resource.error is ErrorResponse ? _resource.error as ErrorResponse : null;
Owner

есть же хелпер toException?

есть же хелпер toException?
Author
Collaborator

toException тут не помогает, он же приводит Object к Exception, но не к ErrorResponse

toException тут не помогает, он же приводит Object к Exception, но не к ErrorResponse
@@ -21,0 +18,4 @@
bool get canResendVerification {
final err = errorResponse;
if (err == null) return false;
switch (err.error) {
Owner

а на код ошибки не надежнее полагаться?

а на код ошибки не надежнее полагаться?
@@ -127,3 +131,4 @@
_lastIntents = null;
_lastPreviewOnly = false;
_lastMetadata = null;
_cancelAutoRefresh();
Owner

лучше все-таки разделить уже провайдер и контроллер. провайдер - про данные, контроллер - GUI-специфичная тема с автообновлением. А то сложно.

лучше все-таки разделить уже провайдер и контроллер. провайдер - про данные, контроллер - GUI-специфичная тема с автообновлением. А то сложно.
protuberanets added 1 commit 2026-02-11 12:26:16 +00:00
tech approved these changes 2026-02-11 15:28:58 +00:00
tech merged commit 3225babeca into main 2026-02-11 15:29:28 +00:00
tech deleted branch SEND047 2026-02-11 15:29:28 +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#464