feat: transactions page #12
No reviewers
malopieds
Labels
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
banquise/laundering!12
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "feat/pending-transactions"
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?
WIP: feat: pending transactions listingto feat: pending transactions listingBon je vais pas review le front a l'oeil nu, et faut que je me remette dedans/relance le projet mais c'est quoi le but de cette PR ?Juste avoir les transactions qui sont en cours ?
@ -24,2 +24,3 @@find("accountId = ?1 AND type != ?2 ORDER BY date DESC, id DESC","accountId = ?1 " +"AND (type != ?2 " + if (!includePending) ") " else "OR type = ?2) " +jcrois jcomprends pas le principe ici, type != ?2 OR type = ?2 dans le cas ou includePending est vrai ?
yep, c est pour eviter une erreur quand je faisais
if (!includePending) "AND type != ?2" else " " else " " +ca mettait une erreur parce que ?2 est pas utilise
@ -30,0 +33,4 @@transactions.page<Transaction>(Page.of(page, size)).list<Transaction>().stream().sorted(Comparator.comparing(Transaction::date).thenComparing(Transaction::createdAt).reversed())sort dans le sql, faire un stream (qui est vraiment une aberration en kotlin vu que c'est java) est trop gourmand alors que sql peut sort pour toi
@ -90,0 +110,4 @@"sum(t.amount) over (ORDER BY t.date ASC, t.createdAt ASC) as balanceAfter, " +"t.accountId, t.createdAt, t.updatedAt, t.`type` " +"FROM Transaction t " +"WHERE t.`type` != ?1 " + if (includePending) "OR t.`type` = ?1 " else "",pareil ici
@ -97,0 +119,4 @@fun getAllTransactionsPaginated(page: Int, size: Int, includePending: Boolean): AccountTransactionEntity {val allTransactions = getAllTransactions(includePending)val transactions = allTransactions.stream<TransactionEntity>().sorted(Comparator.comparing(TransactionEntity::date)pareil ici utilise le sql natif, voir hibernate
yep i know, faut utiliser justement des trucs que j ai pas trop compris, mais le sort des
ORDER BYdans les query comme y a partout a pas l air de marcher, j regarderai mais + tard@ -45,3 +45,3 @@@GET@RolesAllowed("\${roles-admin},\${roles-user},\${roles-clubs}")@Path("/account/{accountId}")@Path("/account/{accountId}/all")le
/allest en trop, transactions est deja au plurielfeat: pending transactions listingto feat: transactions pageView command line instructions
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.