kwila/kalzar
Watch
1
0
Fork
You've already forked kalzar
 0
Code Pull requests Releases Activity Actions

refactor(frontend): introduce data store injection boundary #16

Merged
addison merged 3 commits from feature/data-store-abstraction into main 2026-08-25 17:49:39 -04:00
Conversation 0 Commits 3 Files changed 18 +257 -141
dev-bot commented 2026-08-25 15:54:05 -04:00
Collaborator
Copy link

Summary

  • introduce a domain-facing DataStore interface and Angular injection token
  • keep the abstraction independent from the in-memory implementation provider
  • rename the mock implementation to InMemoryDataStore
  • add an initialization lifecycle and async persistence commands for future remote stores
  • move client, contractor, and job address/contact persistence behind aggregate save commands
  • update application components and tests to use the new boundary
  • keep PocketBase implementation out of scope for a follow-up PR

Validation

  • npm run lint --prefix frontend
  • npx ng build --no-progress (from frontend)
## Summary - introduce a domain-facing `DataStore` interface and Angular injection token - keep the abstraction independent from the in-memory implementation provider - rename the mock implementation to `InMemoryDataStore` - add an initialization lifecycle and async persistence commands for future remote stores - move client, contractor, and job address/contact persistence behind aggregate save commands - update application components and tests to use the new boundary - keep PocketBase implementation out of scope for a follow-up PR ## Validation - `npm run lint --prefix frontend` - `npx ng build --no-progress` (from `frontend`)
refactor(frontend): introduce data store injection boundary
Some checks failed
Pull Request CI / Frontend checks (pull_request) Failing after 44s
Details
Pull Request CI / End-to-end tests (pull_request) Successful in 1m12s
Details
Pull Request CI / Backend checks (pull_request) Successful in 2m24s
Details
ac0504640e 
Amp-Thread-ID: https://ampcode.com/threads/T-01a0393a-f082-7356-be20-5677f672ccd8
Co-authored-by: Addison Emig <[email protected]>
refactor(frontend): prepare datastore for async backends
Some checks failed
Pull Request CI / Frontend checks (pull_request) Failing after 44s
Details
Pull Request CI / End-to-end tests (pull_request) Successful in 1m19s
Details
Pull Request CI / Backend checks (pull_request) Successful in 2m19s
Details
23b916b121 
Amp-Thread-ID: https://ampcode.com/threads/T-01a0393a-f082-7356-be20-5677f672ccd8
Co-authored-by: Addison Emig <[email protected]>
style(frontend): format datastore test provider
All checks were successful
Pull Request CI / Frontend checks (pull_request) Successful in 1m23s
Details
Pull Request CI / Backend checks (pull_request) Successful in 2m15s
Details
Pull Request CI / End-to-end tests (pull_request) Successful in 1m4s
Details
690f225b31 
Amp-Thread-ID: https://ampcode.com/threads/T-01a0393a-f082-7356-be20-5677f672ccd8
Co-authored-by: Addison Emig <[email protected]>
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
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/kalzar!16
No description provided.