SynLotto/website
1
0
Fork 0
Code Issues 19 Pull Requests Actions Packages Projects 5 Releases Wiki Activity

Storage Layer Refactor (No Globals, Query Isolation) #3

New Issue
Open
opened 2025-10-23 21:39:51 +00:00 by H3ALY · 0 comments
H3ALY commented 2025-10-23 21:39:51 +00:00
Owner
Copy Link

Refactor DB init

  • Replace models.SetDB and global DB with storage.Open(cfg).

Context adoption

  • Ensure all storage functions accept context.Context.

Move SQL into local repos

  • Create internal/storage/mysql//repo.go + queries.go.

Embed SQL (optional)

  • Use //go:embed for .sql files per feature.

Acceptance Criteria

  1. No global DB variables.
  2. Storage only exposes constructor + methods.
  3. SQL centralized and reusable.
**Refactor DB init** - Replace models.SetDB and global DB with storage.Open(cfg). **Context adoption** - Ensure all storage functions accept context.Context. **Move SQL into local repos** - Create internal/storage/mysql/<feature>/repo.go + queries.go. **Embed SQL (optional)** - Use //go:embed for .sql files per feature. **Acceptance Criteria** 1. No global DB variables. 2. Storage only exposes constructor + methods. 3. SQL centralized and reusable.
H3ALY added this to the 🛠 In Progress project 2025-10-23 21:39:51 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
backend
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 feature
frontend
help wanted
Need some help
 high-priority
invalid
Something is wrong
 low-priority
question
More information is needed
 security
UX
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project 🛠 In Progress
Assignees
Clear assignees
H3ALY
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SynLotto/website#3
Delete Branch "%!s()"

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?