Sorted based on the number of comments given to others' PRs, but also showing comments on own PRs and other comments given.
[This page was last updated on Nov 15 2020]
@tototto
(1 comments)1 (commented on others PR)
@ZhengShijieNUS @pigoliver @JanuariusJang
Should we place the entire switch case statement in one TRY block & all catch statement after the TRY block for cleaner and more readable code design?
2 (other comment)
Pushed pull request to implement #13
Closing this issue.
3 (other comment)
Fixed the issue in pull request : Added LIST functionality #8
4 (other comment)
Hi Jian Cheng,
I'm fine with integer values too. if everyone is okay let us close this issue and update our user story on fri. 02 oct 2020.
@ZhengShijieNUS @JanuariusJang @pigoliver
5 (other comment)
@pigoliver
I believe we can close this issue. This is because we are following the given user format as per our design:
E.g. add t/cannot submit s/assigned sv/high dl/20Sep2020 o/Jack
Are you agreeable to this?
6 (other comment)
Resolved MERGED CONFLICT in this PR with conflict created in ShiJie commit PR#38 (that has now been merged into Master)
7 (other comment)
Completed PPP
@JanuariusJang
(0 comments)1 (other comment)
Implemented the Delete function that is specified with the index number.
2 (other comment)
JUnit testing for delete function.
3 (other comment)
Added logging, assertions and exceptions on Delete Function
4 (other comment)
looks great
@Yiheng0410
(0 comments)1 (other comment)
done, if start date is later than due date, program will prompt error msg instead of create the project, so the duration also will not be negative.
@Chilaiping
(0 comments)1 (other comment)
Unsuccessful
2 (other comment)
Function has been tested.
3 (other comment)
Check style issue.
@e0260222
(0 comments)1 (other comment)
Failed Gradle Check
2 (other comment)
Failed Gradle check
3 (other comment)
Failed Gradle checks
4 (other comment)
This task is now obsolete.
@skyventus
(0 comments)1 (other comment)
Is it failing the test case
2 (other comment)
Report generation will include this feature.
@daiweinus
(0 comments)1 (other comment)
close #82
2 (other comment)
close #112
@dgc5213
(0 comments)1 (other comment)
Add List Round1. thanks
2 (other comment)
Please help to close this issue. Thanks
3 (other comment)
Please help to close this issue. Thanks
4 (other comment)
Please merge and close. thank you!
5 (other comment)
@e0260222 Hi Weizhong, Please help to merge this push and close #113 Thanks
6 (other comment)
@e0260222 Hi Weizhong, Please help merge it. thanks
7 (other comment)
@e0260222 Hi Weizhong, I did a change to the file. Please help merge it. thanks
8 (other comment)
@e0260222 I'm Sorry for the inconvenience. here's another one needs to merge. thanks
9 (other comment)
@e0260222 Hi Weizhong, I did a change to the image. thanks
10 (other comment)
@e0260222
@e0261618
(0 comments)1 (other comment)
build fail
@adi-kd0021
(0 comments)1 (other comment)
Testing successful
2 (other comment)
Search functionality has been added.
3 (other comment)
Delete function added
4 (other comment)
Add functionality added.
@binbinhui
(0 comments)1 (other comment)
As a project manager, I want to create a project so that I can put in the project information.
2 (other comment)
As a project manager, I wish to be able to add a start and end date to my project so that I know the timeframe.
3 (other comment)
As a project manager, I want to be able to add employees to the project so that I know who are involved.
4 (other comment)
As a project manager, I want to add in the person-in-charge so that I know who is leading the project.
5 (other comment)
As a project manager, I want to add the description to the project so that I know what it is about.
6 (other comment)
As a project manager, I want to be able to add a status of my project so that I will know the progress of this project.
7 (other comment)
This is been done
8 (other comment)
It can be still enhanced on the current logic
9 (other comment)
It can be enhanced by using date/time format
10 (other comment)
This is done
11 (other comment)
Not yet implement can add in under creating project class.
12 (other comment)
This is done but will still leave it open because when more command line more help will be add
13 (other comment)
This is done
14 (other comment)
Skeleton is created.
15 (other comment)
This feature is done
16 (other comment)
Replacement is be done
17 (other comment)
This is done
18 (other comment)
This is done
19 (other comment)
This is done
20 (other comment)
This is done
21 (other comment)
This is done
22 (other comment)
This is done
23 (other comment)
This is done
24 (other comment)
This is done
25 (other comment)
date formate has been check. can improve by using different date formate
26 (other comment)
email command has been added.
27 (other comment)
alert notification has been added. if duration less than 7 days will auto send a email
28 (other comment)
This is done. Test pass
29 (other comment)
This is done.
30 (other comment)
done.
31 (other comment)
closed
32 (other comment)
done
33 (other comment)
Done
@linqing42
(0 comments)1 (other comment)
this is done
2 (other comment)
can be edit in next version
3 (other comment)
this done
4 (other comment)
done
5 (other comment)
I have done and u can pull from upstream and update the Duration such as there are no days left when the status is completed.
6 (other comment)
I have been updated the v2 user stories in DG and add in Complete diagram in our UML class diagram
7 (other comment)
there is a bug in UITest and remember to put the latest version such as include client and email address
@ZhengShijieNUS
(0 comments)1 (other comment)
I agree, too.
@pigoliver
(0 comments)1 (other comment)
Hi hi, the severity was assigned as integer in Defect Class, thus, it need to be an Integer, should we change it to String instead?
2 (other comment)
Resolved with ADD command
3 (other comment)
Feature added in Defect Class.
4 (other comment)
I am agree, waiting for others. Thanks thanks
5 (other comment)
Agree, thanks.
6 (other comment)
Successfully merged all related PR, this issue is closed.
7 (other comment)
Successfully merged all related PR, this issue is closed
8 (other comment)
Successfully merged all related PR, this issue is closed