[Foo Jing Yi] iP #449

foojingyi · 2020-08-27T10:02:30Z

No description provided.

…nd types

…t output

…r Message objects

…ve order

…tialised with the whole of the next line of user input and not with each word of the line as separate objects

…ons to replies to LIST command

…rt of Task's string representation and not added in Message's reply() method

…xample given

… commands, added TODO, DEADLINE and EVENT to Command types

…eply() for NOT_FOUND command

… test

…le exceptions as named in the file names

…nputs with no commands and 2. todo/deadline/event commands with no description, are handled as exceptions

…m "done" and "delete" commands

…WrongItemIndexException

# Conflicts: (Resolved) # src/main/java/duke/Storage.java # src/main/java/duke/TaskList.java

Improve Code Quality

* reduce arrowhead code by abstracting methods out * add new exceptions * change exception names for readibility

Searches for task using the "Find" command can now include multiple keywords and can tasks can be found even if only matching partially.

…ank line above

damithc and others added 30 commits July 23, 2020 23:27

Add Gradle support

3b19ba1

Add Message.java with constructor, reply() method, and enum for comma…

77511f4

…nd types

Added getCmd() method to return command

6c96bd9

Created user input feature and respond to user inputs, deleted defaul…

2777c68

…t output

Changed ECHO command type to ADD with appropriate ouput

09e2eb0

Added a HashSet "list" to keep all added items

52872bf

Added HashSet "list" as variable in Message and to the constructor fo…

e8d0b49

…r Message objects

Change HashSet instances to List implemented with ArrayList to preser…

8cd1a6a

…ve order

Updated reply() method for ADD command to add to list

8e532e7

Change sc.next() to sc.nextLine() so that new Message objects and ini…

2f575a1

…tialised with the whole of the next line of user input and not with each word of the line as separate objects

Added Command type LIST and reply() case for this type

3d27ce2

Updated chatbot's tone in replies

bfad948

Added Task.java class, adapting from example given

a12d247

Changed list from list of String to list of Task, and added status ic…

33bf0df

…ons to replies to LIST command

Added Command type DONE and updated reply() method for this case

c7910a9

Overrode toString method for Task objects such that status icon is pa…

bfb4cbd

…rt of Task's string representation and not added in Message's reply() method

Added Deadline, Event and Todo classes extending Task, adapted from e…

d68bc56

…xample given

Changed accessibility for "private" variables to "protected"

6139e1e

Changed ADD Command type to NOT_FOUND for inputs that don't match any…

86f01bb

… commands, added TODO, DEADLINE and EVENT to Command types

Changed switch statements to if statements in Message.java, updated r…

ebd9ad3

…eply() for NOT_FOUND command

Fixed bug caused by typo in Event class, changed "dl" to "time"

0a72af0

Added reply() method for cases of Command types TODO, DEADLINE and EVENT

3381f40

Updated files in text-ui-test to use the I/O redirection technique to…

839111a

… test

Added DukeException.java to handle exceptions from the chatbot.

de39330

Added NoCommandException.java and NoDescriptionException.java to hand…

758a8e4

…le exceptions as named in the file names

Updated Duke.java and Message.java such that incorrect inputs of 1. i…

2a5ce27

…nputs with no commands and 2. todo/deadline/event commands with no description, are handled as exceptions

Added WrongItemIndexException.java to deal with user input errors fro…

04e0bbe

…m "done" and "delete" commands

Changed exceptions caught in main class to specifically DukeExceptions

086487c

Changed errors arising from using "done" commands to be handled with …

f677798

…WrongItemIndexException

Added DELETE command type and its corresponding reply() method case

b285f63

FooJingYi and others added 30 commits September 10, 2020 18:48

Merge branch 'master' into branch-A-CodeQuality

740f1eb

# Conflicts: (Resolved) # src/main/java/duke/Storage.java # src/main/java/duke/TaskList.java

Merge pull request #2 from foojingyi/branch-A-CodeQuality

21f3d42

Improve Code Quality

Merge branch 'master' of https://github.com/foojingyi/ip

7298c01

Improve code quality

9745094

* reduce arrowhead code by abstracting methods out * add new exceptions * change exception names for readibility

Move reply strings to Ui instead of within command objects

b819617

Specify errors for AddCommand

f280b79

Set up CI

b0f45d7

Set up CI from GitHubActions

314b8a6

Remove wrongly put .yml file

a0e1081

Implement more flexible search

5ce9a68

Searches for task using the "Find" command can now include multiple keywords and can tasks can be found even if only matching partially.

Create gui package

523f0c3

Resolve bug where the first task in a list is added to file with a bl…

6c97df7

…ank line above

Resolve bug where file cannot be loaded

a6a88dd

Make GUI application quit Exit command

487312c

Organise import statements for Ui.java

528ac62

Change file on disk to save to.

ae1e474

Commit MANIFEST.MF file

33db2f0

Rename bot from duke to meimei and changed bot profile picture in GUI

cca9d95

Refactor exception classes names and variable names to fit with Meimei

019a2e6

Improve coding style according to coding standards

b535320

Change user profile and changed mentions of duke to meimei

4037a86

Change format of the reader friendly date and times

70f88c2

Add Ui.png to docs folder

b85a7da

Update README for Meimei Bot

b47957c

Set theme jekyll-theme-architect

86334ec

Update build.gradle

b545ebc

Shorten line in MainWindow.java to fit with coding conventions

c8beb1f

Delete unnecessary test class and fix import statement order

215ce02

Change to separate imports in TastTest.java

907230e

Fix files according to date time format

ee28f71

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Foo Jing Yi] iP #449

[Foo Jing Yi] iP #449

foojingyi commented Aug 27, 2020

[Foo Jing Yi] iP #449

Are you sure you want to change the base?

[Foo Jing Yi] iP #449

Conversation

foojingyi commented Aug 27, 2020