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

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 +172 -121
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.
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 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 labels
bug
duplicate
enhancement
help wanted
invalid
question
techdebt
wontfix
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
personal-finance/datastore!45
No description provided.