Redefine models

[kungpaogao]

We need to redefine our models

General

• need to update/add more fields to enable quicker identification of furniture
• need to update how we type check in TypeScript (interface vs. type vs. classes)
  • see this Stack Overflow post: Difference between interfaces and classes in Typescript
• we don't want our app to crash b/c we stored something bad to the database
• add tag attribute for type checking
• use schema attribute so we can query on specific versions of interfaces/classes

Furniture

• add name/desc field (quick reference) - there was some discussion on re-purposing the comments section
• separate bed components (frame, mattress, ends) under FClass

Donor

• split up address: street address, city, state, zip code - this is mainly to make edit UI more intuitive and can be used for filtering
• remove zone

Timing

• consider Firebase's usage of Timestamp type
• temporarily addressed in commit acf2d88
• this created some issues with Batch update to furniture in runs breaks Date/Timestamp objects #74 I think - addressed here a25bf30

[kira-segenchuk]

update everyone if you make changes here

[kungpaogao]

if we have optional fields, we need to decide how to handle

[kungpaogao]

Updated original comment with more changes

[kungpaogao]

Update original comment for changes regarding Timing and a link to this nice post:
https://stackoverflow.com/questions/40973074/difference-between-interfaces-and-classes-in-typescript

Here's another related post: https://stackoverflow.com/questions/51716808/when-use-a-interface-or-class-in-typescript

[kungpaogao]

We should also consider moving data (the folder that contains all the types) to the root of our project so that it's clear that our Firebase Functions should be able to access them.

[kungpaogao]

In terms of migrating our models, I think a good way of doing it is by adding a version (or schema_version) to our model. This way we can just query based on the schema version to prevent our frontend from breaking / having inconsistent models.

[kungpaogao]

After some discussion and consideration, I think the move might be to go with TypeScript interfaces over classes.

[kungpaogao]

A good(?) idea by @kira-segenchuk could be having a global tag property that we can use for "type-checking"