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

New Firestore Schema with Subcollections #270

Merged
merged 28 commits into from
Nov 24, 2023
Merged

New Firestore Schema with Subcollections #270

merged 28 commits into from
Nov 24, 2023

Conversation

mikaelacaron
Copy link
Owner

@mikaelacaron mikaelacaron commented Nov 23, 2023 •
edited
Loading

What it Does

How I Tested

  • Make sure the app still works with all CRUD functions

Notes

  • N/A

Screenshot

  • N/A

mikaelacaron and others added 28 commits November 20, 2023 21:06
@mikaelacaron
Fix spelling
fb9cd15
@mikaelacaron
Move functions inside of AuthenticationViewModel
a7c3228
@mikaelacaron
Save maintenance event as a subcollection of vehicle
db1bc7d
@mikaelacaron
DashboardViewModel
48007eb
@mikaelacaron
Use composite query to load maintenance events
ff50cdc
@mikaelacaron
Move analytics event
9ec3f29
@mikaelacaron
Update an event
4d77a06
@mikaelacaron
Can't change vehicle when updating an event
5df5c62
@mikaelacaron
Show the vehicle name
8a7d9f6
@mikaelacaron
Update path for delete maintenance event
4d86bbb
@mikaelacaron
Create FirestorePath enum
e6aa92f
@mikaelacaron
Make all GitHubURLs into an enum
a8f5572
@mikaelacaron
Add image for DocC about Firestore
1860ea8
@mikaelacaron
Create an odometer reading using sub-collections
0a88a4f
@mikaelacaron
Fix whereField for getting maintenance events
90f5875
@mikaelacaron
Get odometer readings with a collection group
ee0e0f7
@mikaelacaron
Show vehicle name properly
bd51b19
@mikaelacaron
By default select a vehicle when adding a maintenance event
7b4f17f
@mikaelacaron
Delete odometer reading
d74c89f
@mikaelacaron
Only pass User UID to DashboardView and VM
32d74ef
@mikaelacaron
Only pass user UID to OdometerView and VM
d9197c2
@mikaelacaron
Comment out deleting a vehicle
ea75476 
Because it requires using a Cloud function, because of the sub-collections
@TratonGossink
5 Edit Vehicle (#269)
a6f96fa
@mikaelacaron
Merge branch 'fix-vehicle-event-connection' of https://github.com/mik…
963d3b0 
…aelacaron/Basic-Car-Maintenance into fix-vehicle-event-connection
@mikaelacaron
Fix some of the edit vehicle views
1f77076 
Fix some logic and variables from PR 269
@mikaelacaron
Fix Localizable.xcstrings
87e57bd
@mikaelacaron
Correct Firebase function to update a vehicle
9ec9ac8
@mikaelacaron
Remove comment
c463788
@mikaelacaron mikaelacaron marked this pull request as ready for review November 24, 2023 22:28
@mikaelacaron mikaelacaron merged commit 1ebd13e into dev Nov 24, 2023
2 checks passed
@mikaelacaron mikaelacaron deleted the fix-vehicle-event-connection branch November 24, 2023 22:44
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.
2 participants
@mikaelacaron @TratonGossink