AY1920S1-CS2103-F09-4 / main

Insurelytics
https://ay1920s1-cs2103-f09-4.github.io/main/
MIT License
1 stars 5 forks source link

Refactor addPerson to take out policy and tag #76

Closed larrylawl closed 5 years ago

larrylawl commented 5 years ago

I realised that our addperson command allows the user to add policies and tags when creating a new contact, but editperson does not allow that, which might be a little confusing for both the user and us. I was thinking that since we already have the commands assignpolicy, unassignpolicy, addtag, deletetag why not just separate the adding/editing of policies and tags entirely and leave that responsibility to the various commands mentioned. This means that addperson and editperson only touches the particulars of the contact, while anything else related to policies and tags will have their own helper commands like the ones raised above. This allows us and our users to draw a distinct line between things related to person, tags, and policies, which i think makes the user workflow simpler. The same goes for addpolicy and editpolicy.