jhachtel / AMIS3610-Group-Project

0 stars 0 forks source link

Review API Models #57

Closed jhachtel closed 5 years ago

jhachtel commented 5 years ago
jhachtel commented 5 years ago

Working on branch api-models-review

jhachtel commented 5 years ago

ApplicationUser is as presented in the lab. No changes made. No changes detected.

jhachtel commented 5 years ago

Gear model has attributes ID, Name, Brand, Type, Description.

Adding Link attribute. (FYI - @Demerti @AlexTetzloff)

jhachtel commented 5 years ago

People model has attributes Id, Name, Contactinfo, and Bio.

(FYI - @Demerti @AlexTetzloff ) Renaming: People model to Person, People can now be a list of Persons elsewhere. string Contactinfo to string Email.

Adding: List<Person> Friends, List<Gear> Gear, List<Place> Places, List<Trail> Trails, List<Trip> Trips.

The above will allow you to populate the Person's page with something interesting.

Edited because it didn't format like I wanted.

jhachtel commented 5 years ago

@Demerti - What do you think of changing gear to item - because of the plurality issue? Could still be displayed as "gear" on the site if you want.

jhachtel commented 5 years ago

(FYI - @Demerti @AlexTetzloff ) Associate and Disassociate methods added to Person model to add and remove from the various lists.

jhachtel commented 5 years ago

GearTest updated to reflect new link field.

jhachtel commented 5 years ago

Person test added, but list tests still need to be built. Holding off for now, opening issue to remind myself.

jhachtel commented 5 years ago

(FYI - @Demerti @AlexTetzloff ) Added link attribute to place.

jhachtel commented 5 years ago

(FYI - @Demerti @AlexTetzloff ) Added link attribute to trail.

jhachtel commented 5 years ago

@Demerti @AlexTetzloff

Models refactored. Look at them so you know what's there. If you need something added, let me know.

Kane, you will need to match the fields in the .ts files. (Gear.ts, Person.ts, etc...)