kwila/office-automations
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Activity Actions

WIP: fix: invoice order #14

Closed
addison wants to merge 3 commits from fix/invoice-order into main
pull from: fix/invoice-order
merge into: kwila:main
Conversation 1 Commits 3 Files changed 5 +81 -12
addison commented 2025-12-01 17:36:54 -05:00
Owner
Copy link
No description provided.
feat: try to improve order
All checks were successful
CI / frontend (pull_request) Successful in 36s
Details
CI / backend (pull_request) Successful in 58s
Details
84e826eef5
chore: add a script to compare
All checks were successful
CI / frontend (pull_request) Successful in 38s
Details
CI / backend (pull_request) Successful in 59s
Details
28ef60da8c
addison reviewed 2025-12-01 18:06:11 -05:00
backend/internal/server/invoices.go
@ -148,4 +148,3 @@
FROM aptCDV_VW_APT_InvSalCredEstList i
LEFT JOIN Invoices inv ON i."Tran No" = inv.InvoiceNumber
` + whereClause + `
ORDER BY i."Tran Date" ASC, i."Tran No" ASC`
addison commented 2025-12-01 18:06:11 -05:00
Author
Owner
Copy link

I think the old order makes more sense. I will wait to get new CSV from Aptora export to make sure I actually need any changes.

I think the old order makes more sense. I will wait to get new CSV from Aptora export to make sure I actually need any changes.
addison changed title from WIP: fix: invoice order to draft: fix: invoice order 2025-12-01 18:06:23 -05:00
addison changed title from draft: fix: invoice order to WIP: fix: invoice order 2025-12-01 18:06:30 -05:00
addison closed this pull request 2025-12-03 15:18:16 -05:00
All checks were successful
CI / frontend (pull_request) Successful in 38s
Details
CI / backend (pull_request) Successful in 59s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
kwila/office-automations!14
No description provided.