McGill-ECSE321-Winter2020 / project-group-01

project-group-01 created by GitHub Classroom
2 stars 5 forks source link

Advertisement class doesn't have a field "Set<Pet> pets" #91

Closed christopher-boustros closed 4 years ago

christopher-boustros commented 4 years ago

According to the domain model, each advertisement should have 1..* pets. Hence, there should be a field for pets in the Advertisement model class.

katrinapoulin commented 4 years ago

I think the association is unidirectional in the umlcd file. Is it useful to make it bidirectional (what would we get from setting pets to advertisement)?

christopher-boustros commented 4 years ago

I was confused because it was bidirectional in the model on the wiki. Issue closed.

Louismousine commented 4 years ago

Misclick

katrinapoulin commented 4 years ago

I will change it in the wiki for Sunday.

Louismousine commented 4 years ago

the deliverable is due saturday bro