-
Notifications
You must be signed in to change notification settings - Fork 4
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.
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
Update Participation class and create corresponding methods in Project #71
Update Participation class and create corresponding methods in Project #71
Conversation
* What changes made: | ||
* added dependencies of it in and related collections; | ||
* added related test cases. | ||
|
||
* **Model modification**: Update Participation class and create corresponding methods in Project. (Pull request: Pull request [\#71](https://github.com/AY2021S1-CS2103T-W10-3/tp/pull/71)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good contribution
* Added documentation for basic task-related features. | ||
* Came up with the first draft of feature list. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can consider linking this to your commit?
* Developer Guide: | ||
* Wrote first draft of non-functional requirements. | ||
* Wrote first draft of non-functional requirements. (Pull Request [\#54](https://github.com/AY2021S1-CS2103T-W10-3/tp/pull/54)) | ||
|
||
* **Community**: | ||
* Updated `README.md` for team repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link this to your commit too? :)
public Participation(Person person, Project project, Role role) { | ||
this.person = person; | ||
this.project = project; | ||
this.role = role; | ||
tasks = new HashSet<>(); | ||
meetings = new HashSet<>(); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good implementation of the Association class, we'll see how this plays out when we build the subsequent features.
No description provided.