Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor to reactive #7

Merged
merged 82 commits into from
Feb 13, 2024
Merged

Refactor to reactive #7

merged 82 commits into from
Feb 13, 2024

Conversation

efuller
Copy link
Owner

@efuller efuller commented Feb 8, 2024

No description provided.

efuller added 30 commits February 4, 2024 10:18
@efuller
Refactor jest config and scripts
d290aa9
@efuller
Initial setup of app level acceptance tests
aec1003
@efuller
Make sure to pass in test arg
3cfcb5f
@efuller
Move journal model into a journal module
81684d1
@efuller
Add in an initial router
c836898
@efuller
Add in an Observable
9ad651b
@efuller
Scaffold initial app internals
2dc5409
@efuller
Check if on the homepage
7058ff0
@efuller
Rename food module to journal
3057f64
@efuller
Only pass the value of the subscriber
6f54524
@efuller
Wire up the presenter to the repo
9eba0ee
@efuller
Wire up the presenter
56cdddf
@efuller
Add an isFavorite property to Journal
e459ac1
@efuller
Add in test for deleting a journal entry
0210284
@efuller
Add in a test for setting a favorite journal and saving it to a favor…
dfa8e40 
…ites repo
@efuller
Add in a test for deleting an entry that is set as a favorite
e54cee9
@efuller
Wire up the presenter to local state and use our controller to add a …
3bb17fb 
…journal entry
@efuller
Wire up deleting a journal entry
856e140
@efuller
Sync with client storage on load
55feafe
@efuller
Create local storage client
4891bf6
@efuller
foodId should be journal
ce9d06d
@efuller
Based on context, create appropriate client storage repo
655ce63
@efuller
Wire up delete and confirm delete on the frontend
ce986f6
@efuller
Refactor confirmation modal into an observable
dfa1f8f
@efuller
Refactor more logic out of App.tsx
219a72e
@efuller
Move confirmationModal.ts into shared
8ac872d
@efuller
Remove code we are no longer using at the moment
3a08f84
@efuller
Consolidate state into one object
9c29927
@efuller
Add in a couple of helpers to the presenter
a8f73ea
@efuller
Clean up the controller
343275b
efuller added 28 commits February 11, 2024 10:00
@efuller
Install jsdom environment
7c7b1c3
@efuller
Configure infrastructure tests
bab5daa
@efuller
Add in a contract test for ClientStorageRepository interface
63e603b
@efuller
Get E2E tests to pass
71c9c23
@efuller
Clean up app tests
61b29bb
@efuller
Add in a testing scenario for hydrating state from client storage on …
48cf452 
…app load
@efuller
Move application level acceptance tests into src/tests folder
7db0d9c
@efuller
Move view model out into its own class
c62a1af
@efuller
Fix imports
c8e9cf6
@efuller
Wire up modal to only show when deleting a pending favorite
ef4994a
@efuller
Remove useConfirmationModal.ts
dcc9046
@efuller
Drive confirmation modal via app state
0c8f7ec
@efuller
Remove vm dependency in the confirmation modal
69d9bb8
@efuller
Refactor out a JournalForm component
5792b6d
@efuller
Extract a JournalList component
6ce021a
@efuller
Use dto rather than form input
625c913
@efuller
Rename folder
296cae9
@efuller
Remove console.log
05d2f72
@efuller
Remove MSW
2622656
@efuller
Update readme.md from Eraser
27cf489
@efuller
Update readme.md from Eraser
4d01117
@efuller
Update readme.md from Eraser
ea2b791
@efuller
Update readme.md from Eraser
8a5f2de
@efuller
Add in a GH Action workflow for PRs
61986cc
@efuller
Merge remote-tracking branch 'origin/refactor-to-reactive' into refac…
603bdb1 
…tor-to-reactive
@efuller
Remove top link
1c3ba6b
@efuller
Update readme.md
d18e88a
@efuller
Remove unused code
084a9fe
@efuller efuller merged commit bca5c52 into main Feb 13, 2024
1 check passed
@efuller efuller deleted the refactor-to-reactive branch February 13, 2024 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@efuller