personal-finance/datastore
2
0
Fork 0
Code Issues 12 Pull Requests Actions Packages Releases Wiki Activity

refactor: swap strings with an SQL builder #45

Merged
satprog merged 2 commits from feat/sql-builder into main 2025-09-05 21:15:21 +00:00
Conversation 0 Commits 2 Files Changed 6 +179 -128
satprog commented 2025-08-24 22:28:28 +00:00
Owner
Copy Link

Adds doug-martin/goqu as dependency.
Make tests less fragile by focusing only on relevant SQL query parts.

refactor: return err from API handlers
Instead of logging + NoContent(http.StatusIntervalServerError).
Gin already logs the error when returned and returns a 500 to the client.

Adds doug-martin/goqu as dependency. Make tests less fragile by focusing only on relevant SQL query parts. refactor: return err from API handlers Instead of logging + NoContent(http.StatusIntervalServerError). Gin already logs the error when returned and returns a 500 to the client.
satprog added 2 commits 2025-08-24 22:28:28 +00:00 
Adds doug-martin/goqu as dependency.
Make tests less fragile by focusing only on relevant SQL query parts.
refactor: return err from API handlers
All checks were successful
Lint PR / commitlint (pull_request) Successful in 28s
Details
Go / build (1.22) (pull_request) Successful in 1m10s
Details
Go / build (1.23) (pull_request) Successful in 1m8s
Details
Go / build (1.24) (pull_request) Successful in 1m11s
Details
Go / build (1.22) (push) Successful in 1m7s
Details
Go / build (1.24) (push) Successful in 1m3s
Details
Go / build (1.23) (push) Successful in 1m5s
Details
547698872d 
Instead of logging + NoContent(http.StatusIntervalServerError).
Gin already logs the error when returned and returns a 500 to the client.
satprog merged commit 547698872d into main 2025-09-05 21:15:21 +00:00
satprog deleted branch feat/sql-builder 2025-09-05 21:15:22 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
techdebt
Technical debt
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
joanaosoriosousa nunorosa satprog
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: personal-finance/datastore#45
No description provided.
Delete Branch "feat/sql-builder"

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?