ay2021s1-cs2113-t14-3 / tp Goto Github PK

Command to delete tasks before current date.

The error I'm getting is the format of the date but I follow the format exactly just that I did not include the argument . So maybe it's better to have another error handling that states exactly what I'm missing.

Labels: severity.Medium type.FunctionalityBug
original: dixoncwc/ped#4

Delete items by category. For example, delete c/cs2113.

As a user i would like to be able to keep track of my tasks.

As a user, i would like to safely exit the program.

As a freshman year 1 NUS student, I want to be able to compute my predicted grades (on top of current grades) and view my remaining SUs, so that I can focus on modules that maximise my CAP.

According to the add module description in the User Guide, it is stated:

However, I think it would be better to further specify what is a valid academic year. If I input “AY2011S1”, which is invalid because A AY2011 does not actually exist. Since the first two digits of 2011 represent the last two digits of a year, and last 2 digits represent another year, it doesn’t really make sense to allow such an academic year to be allowed. the app would’ve been better if it verifies if the user is adding a correct academic year first.

Labels: severity.Low type.FeatureFlaw
original: acyang97/ped#4

Maybe you can include checking of inputs in lower case as some user might not use CAPS to enter input.

Labels: severity.VeryLow type.FunctionalityBug
original: dixoncwc/ped#2

As a student, I want to assign priority to tasks so that I can focus on the important things in life.

As a first time user, it would be helpful to have a help command which lists all the available commands and format allowed.

As a user, I want to log in with a password so that nobody else using my laptop can access my data.

date <taskIndexNumber> date/<dd-MM-yyyy> sets the date of a given task.

As a user i would like to have a feature to delete a task.

The argumentsMap will replace value p with the second value if the user passes in 2 of the same arguments.

For example, add tP meeting p/1 p/2 will set the priority to 2. This is because argumentsMap.get("p") returns "2" and the arguments are parsed sequentially.

Fix: when parsing arguments, check if "p" already exists in the argumentsMap. Throw an error if the user passes in duplicate arguments.

Personally, I think this line is quite vague. I suggest to state “Lists all the tasks including recurring task” instead.

Labels: severity.VeryLow type.DocumentationBug
original: acyang97/ped#8

As a student, I want to be able to save all the Zoom meeting links in a file so that it will be easy for me to find.

As a user I can so that I can retrieve my previously saved data next time when I open the program.

• list
• list a

As a user I can so that I can keep track of the categories of my tasks.

Example: add tP meeting recurring/05-10-2020:15-11-2020:wednesday will add this task for every wednesday between 05-10-2020 to 15-11-2020

ps: maybe we can probably decide on a better command format or this may be fine

List tasks by category. For example, list c/cs2113.

As a student and library user, I want to keep track of my borrowed items and have the return date show up on the calendar, so that I can prevent myself from incurring library fines.

I think the format should be written as addr s/ e/, etc ... to be clearer and more easily understood by the user

Labels: severity.Low type.DocumentationBug
original: acyang97/ped#1

As a user, I would like to have all my tasks listed, so I would have a better view of things I need to do.

Update User Guide for v2.0

Our Java CI is failing due to IO redirection tests. Try to fix after merging code from the iP.

As a student, I want to keep track of my daily, weekly, and monthly expenses so that I can better manage my money.

For the find method, it was stated in the UG that this method searches tasks with a keyword. I think it would be better to specify what your group meant by a keyword. The method works, but if a user happen to type b as the ‘keyword’, the app returns a list of tasks as long as it contains the character ‘b’ in the description. To me or others, keywords for that task might be “board”, “games” and “club”, and typing “find b” should not have returned the following tasks as shown in the picture below.

Labels: severity.Medium type.FeatureFlaw
original: acyang97/ped#11

As a student who lives on campus, I want to be able to keep track of my remaining meal credits, so that I can better manage my breakfasts and dinners.

Might be good if you can ensure that no two modules of the same code and same AY to the list of modules. :)

Labels: severity.Low type.FeatureFlaw
original: acyang97/ped#3

I think it would be good to let users know that for this function, the category is case-sensitive.

Labels: severity.Low type.DocumentationBug
original: acyang97/ped#9

calendar x prints the tasks for the next x days.

example command: calendar 3
example output:

---tuesday Oct 20 2020---
task 1
task 2
---wednesday Oct 21 2020---
no tasks on this date
---thursday Oct 22 2020---
task
task
-----------------------------

I think it would be better to indicate where these folders are located. The locations might not be as straightforward for users who have little knowledge on folder directories.

Labels: severity.Low type.FeatureFlaw
original: acyang97/ped#7

Seems like should been add task tP meeting c/cs2113 for the first one, missing the word task. Similarly for the second one as well.

Labels: severity.Medium type.DocumentationBug
original: acyang97/ped#10

I tried to add an recurring tasks but wrote the wrong start date and end date but the error that it prompt is different from what it supposed to be detecting. Maybe you can adjust the error handling.

Labels: severity.Medium type.FunctionalityBug
original: dixoncwc/ped#5

Just a suggestion, for the add module, although it works, I think it would be better to set a limit on the number of mc for a module to make your app more logical and catered to NUS students.

Not very sure if there are 0mc modules in NUS too, you might want to double check.

Labels: severity.Low type.FeatureFlaw
original: acyang97/ped#2

Maybe you can adjust the layout to make it consistent so that user can visualize it better

Labels: severity.Low type.DocumentationBug
original: dixoncwc/ped#3

I think it might be better to explain in the UG what is priority. For instance, if I want to set a task to have a higher priority, do I give it a higher priority integer value or lower?

Labels: severity.VeryLow type.DocumentationBug
original: acyang97/ped#13

As a user i would like to delete tasks based on the priorities.

As a student, I want to list my tasks sorted by priority so that I can know what tasks to work on first.

CommandCreator class can create commands with code separately instead of putting all the code for creating commands within the Parser class.
https://se-education.org/se-book/designPatterns/#command-pattern

I think it might be better to either state in the UG that the due date will be by default 1 month after the starting date or allow the user to actually indicate the due date to make it clearer to the user.

Labels: severity.Low type.DocumentationBug
original: acyang97/ped#6

Keep track of dates so that users can print a weekly calendar, etc.

Example command:

• add tP meeting date/2020-10-13

It would be good if in general for all the methods in the UG, the group states the input is case-sensitive or case-insensitive to be consistent throughout the UG. Would also be better for the user to know how to use the app.

Labels: severity.VeryLow type.DocumentationBug
original: acyang97/ped#14

According to the UG, it seems linke only a URL is a valid input as the link.

However, by trying out using any random input that may not necessarily be a url link, function still works. I think it might be better to somehow enforce that only a link is allowed as an input, else the functionality of 'Adding a weblink' might not be as perfect as people want it to be.

Labels: severity.Medium type.FeatureFlaw
original: acyang97/ped#5

As a student, I want to be able to automatically create file directories and folders on my computer (i.e. 2021S1 → CS2113 and this folder contains ‘Tutorial’ and ‘Lecture Notes’ folders) for the modules that I am taking this semester, so that I do not have to manually create them.

I personally think that thepriority that the user can state could have been better utilized. I don’t think deleting based on the priority level is really necessary. If a user has many different tasks (lets say 1000), it would be difficult for him to ensure that the tasks of a certain priority level in the list are the ones he really want to delete. Hence, I suggest coming up with better ways to make use of the priority, such as ordering the list of tasks using dates followed by priority, which might be possible using a Comparator or Comparable.

Labels: severity.Low type.FeatureFlaw
original: acyang97/ped#12

In the user guide the screenshot output differ from the actual output I'm getting. Might be either a documentation bug in user guide or coding error in printing the value.

Labels: severity.VeryLow type.DocumentationBug
original: dixoncwc/ped#1

As a user, i would prefer to have an option to delete all tasks as well.