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

added download for operation and included fixes for source of payments #639

Merged
tech merged 3 commits from SEND063 into main 2026-03-05 08:29:46 +00:00
Conversation 4 Commits 3 Files Changed 59 +1389 -531
protuberanets commented 2026-03-04 14:45:29 +00:00
Collaborator
Copy Link
No description provided.
protuberanets self-assigned this 2026-03-04 14:45:30 +00:00
protuberanets added 1 commit 2026-03-04 14:45:31 +00:00
SEND063 aff804ec58
protuberanets requested review from tech 2026-03-04 14:45:31 +00:00
protuberanets added 1 commit 2026-03-04 15:07:21 +00:00
updated document upload according to fresh api c59538869b
tech reviewed 2026-03-04 17:04:03 +00:00
frontend/pshared/lib/data/dto/payment/operation.dart
@@ -0,0 +23,4 @@
this.completedAt,
});
factory PaymentOperationDTO.fromJson(Map<String, dynamic> json) =>
tech commented 2026-03-04 17:04:03 +00:00
Owner
Copy Link

Давай на JSON serialisable перейдем

Давай на JSON serialisable перейдем
tech reviewed 2026-03-04 17:08:48 +00:00
frontend/pweb/lib/utils/report/payment_mapper.dart
@@ -53,0 +65,4 @@
final pair = parseOperationCodePair(operation.code);
if (pair == null) continue;
if (pair.operation != 'card_payout' || pair.action != 'send') continue;
tech commented 2026-03-04 17:08:48 +00:00
Owner
Copy Link

вот, говорил: вот эту штуку хардкодить не надо, лучше вынести куда-то в решающее правило, а не запаковывать внутрь провайдера, где найти ее непросто.

вот, говорил: вот эту штуку хардкодить не надо, лучше вынести куда-то в решающее правило, а не запаковывать внутрь провайдера, где найти ее непросто.
tech requested changes 2026-03-04 17:10:52 +00:00
Dismissed
frontend/pweb/lib/widgets/payment/source_wallet_selector.dart Outdated
@@ -157,6 +130,20 @@ class SourceWalletSelector extends StatelessWidget {
_SourceOptionKey _ledgerKey(String ledgerAccountRef) =>
(type: PaymentSourceType.ledger, ref: ledgerAccountRef);
String _walletDisplayName(Wallet wallet, AppLocalizations l10n) {
tech commented 2026-03-04 17:10:13 +00:00
Owner
Copy Link

дублирование кода: по-хорошему нужна внутренняя функция-хелпер, которая реализует одну логику для разных вводных. А верхнеуровневые функции уже предоставляют удобный интерфейс с сокращенным списком параметров.

дублирование кода: по-хорошему нужна внутренняя функция-хелпер, которая реализует одну логику для разных вводных. А верхнеуровневые функции уже предоставляют удобный интерфейс с сокращенным списком параметров.
protuberanets added 1 commit 2026-03-04 22:49:00 +00:00
few fixes and made sure ledger widget displays the name of ledger wallet 519a2b1304
tech approved these changes 2026-03-05 08:29:39 +00:00
tech merged commit 364731a8c7 into main 2026-03-05 08:29:46 +00:00
tech deleted branch SEND063 2026-03-05 08:29:46 +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#639
Delete Branch "SEND063"

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?