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
Implemented AddMilestoneCommandParser #48
Conversation
… & DG) Updated UG and DG for addTask + addDeadline
# Conflicts: # docs/DeveloperGuide.adoc # docs/UserGuide.adoc
# Conflicts: # docs/DeveloperGuide.adoc
# Conflicts: # docs/DeveloperGuide.adoc
Changed "rank" type from Integer to String in the Milestone class
Fixed minor checkstyle issues
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.
Please pull again, this is not the latest version.
super(title, milestoneDescription, new PriorityLevel("high")); | ||
//this.title = title; | ||
//this.milestoneDescription = milestoneDescription; | ||
this.rank = rank; | ||
} | ||
|
||
public Integer getRank() { | ||
public String getRank() { |
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.
More intuitive to have rank as a int? U may consider parsingRank as a string then converting it into an integer.
…into v1.3_Jeremy
This reverts commit fe913e4.
…rent list of tasks and add milestones to that particular task
@@ -15,13 +17,15 @@ | |||
private final String description; | |||
private final PriorityLevel priorityLevel; | |||
private boolean isCompleted; | |||
private final Set<Milestone> milestoneSet; |
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.
Setting milestoneSet to final means you cannot add new milestones to it, or change deadlines in the milestones
Milestone Class: Changed "rank" type from Integer to String
Updated LogicManager
Fixed minor checkstyle issues