-
Notifications
You must be signed in to change notification settings - Fork 78
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
[lckjosh] iP #74
Open
lckjosh
wants to merge
37
commits into
nus-cs2113-AY2324S1:master
Choose a base branch
from
lckjosh:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,106
−37
Open
[lckjosh] iP #74
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
9af57b0
add greeting feature and exit
lckjosh f35e5e1
rename bot to BotBuddy
lckjosh c4e3f6e
add echo feature
lckjosh d219a1f
add ability to add items and list them
lckjosh 2d8c682
create Task class
lckjosh 663cf7c
add ability to mark and unmark tasks as done
lckjosh 87a3330
follow coding standard
lckjosh e4200c5
categorise tasks into Todo, Event, and Deadline
lckjosh 2a749bc
create methods for each command and call the methods in switch statement
lckjosh 8060406
use do-while loop
lckjosh bd0006b
add input and output testing
lckjosh 1d647e3
handle unrecognised commands
lckjosh 9e488c2
create BotBuddyException exception
lckjosh e2e415d
add error handling
lckjosh 704093c
Merge branch 'branch-Level-5'
lckjosh 00bb9a7
put classes in BotBuddy package
lckjosh a18d4c0
Merge branch 'branch-A-Packages'
lckjosh 3223c2a
rename Duke.java to BotBuddy.java
lckjosh 5637b71
use ArrayList<Task> to store Tasks instead of an array
lckjosh e1dee36
add ability to delete tasks
lckjosh 713b1a3
Merge branch 'branch-Level-6'
lckjosh 8254280
add ability to save and read from file
lckjosh 995547b
auto create task file and directory if not found
lckjosh 8e41dc0
Merge branch 'branch-Level-7'
lckjosh 722041b
remove duplicate check if directory is created
lckjosh a9e8f7d
update gitignore
lckjosh a2c02d8
refactor code to make it more OOP
lckjosh f95d5a1
add ability to find tasks
lckjosh 3957ae9
Merge pull request #1 from lckjosh/branch-Level-9
lckjosh 2868ee6
updated list of supported commands
lckjosh 1ec406b
update README
lckjosh ff1c2ca
fix READMEs
lckjosh 51145c6
add javadocs
lckjosh 690e241
edit javadoc
lckjosh 59e9b5e
Merge pull request #2 from lckjosh/A-JavaDoc
lckjosh 54dd0f9
edit javadocs
lckjosh 63a2454
Merge pull request #3 from lckjosh/branch-A-JavaDoc
lckjosh File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
categorise tasks into Todo, Event, and Deadline
commit e4200c5476d64a228206c6572516de62caaae824
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
public class Deadline extends Task { | ||
|
||
protected String by; | ||
|
||
public Deadline(String description, String by) { | ||
super(description); | ||
this.by = by; | ||
} | ||
|
||
@Override | ||
public String toString() { | ||
return "[D]" + super.toString() + " (by: " + by + ")"; | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
public class Event extends Task { | ||
|
||
protected String from; | ||
protected String by; | ||
|
||
public Event(String description, String from, String by) { | ||
super(description); | ||
this.from = from; | ||
this.by = by; | ||
} | ||
|
||
@Override | ||
public String toString() { | ||
return "[E]" + super.toString() + " (from: " + from + " to: " + by + ")"; | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
public class Todo extends Task { | ||
public Todo(String description) { | ||
super(description); | ||
} | ||
|
||
@Override | ||
public String toString() { | ||
return "[T]" + super.toString(); | ||
} | ||
} |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Consider using a constant / final variable to represent the line divider