Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Pontakorn Prasertsuk] iP #25

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

[Pontakorn Prasertsuk] iP #25

wants to merge 60 commits into from

Conversation

peppapighs
Copy link

@peppapighs peppapighs commented Aug 19, 2022 •
edited
Loading

Bob Chatbot

“Your mind is for having ideas, not holding them.” – David Allen (source)

Bob Chatbot helps you remember your tasks. It's,

  • text-based
  • easy to learn/use
  • blazingly fast!

To use, simply following these steps

  1. Download the jar file from here
  2. Double click the jar file to open the program
  3. Add your tasks
  4. Let the bot manage your tasks!

and it's FREE

Features:

  • Managing tasks
  • Managing deadlines (coming soon)
  • Reminders (coming soon)

damithc and others added 30 commits July 31, 2022 17:20
@damithc
Add Gradle support
556af3f
@peppapighs
ignore .class file
0541256
@peppapighs
Level-1: greeting text & reading commands
e74f591
@peppapighs
read input as line
94c4aef
@peppapighs
Level-2: add items to the list
5e21c50
@peppapighs
add task class with status
d95c6ca
@peppapighs
Level-3: mark task as done and not done
6f3cc18
@peppapighs
split input into arguments
ea9ea5c
@peppapighs
add todo & deadline tasks
562ba72
@peppapighs
Level-4: add event task
89d72a0
@peppapighs
A-TextUiTesting: add test case
d6a81e7
@peppapighs
Level-5: handle possible errors
6b87e2a
@peppapighs
Level-6: delete task
f932a2a
@peppapighs
Format code with google-java-style
ddaa77c
@peppapighs
Refactor bot main logic
00bcfb9
@peppapighs
Use regex for splitting command
ba03ba8
@peppapighs
Add save & load tasks from file
41d913b
@peppapighs
Format date with LocalDate
df38ec4
@peppapighs
Merge branch 'branch-Level-8'
d636831
@peppapighs
Extract & package classes
fe2a5f4
@peppapighs
Merge branch 'add-gradle-support' of https://github.com/nus-cs2103-AY…
0f776df 
…2223S1/ip
@peppapighs
Add Command classes for executing commands
f2e4507
@peppapighs
Refactor try-catch in Parser
882e862
@peppapighs
Rename exception package to common package
bcffb7f
@peppapighs
Add encode test for Task
c234e5b
@peppapighs
Fix task decode bug
bd8637b
@peppapighs
Show numeric order for task list
b30c497
@peppapighs
Add TaskList & Parser tests
6911b06
@peppapighs
Configure gradle main class name
da8202e
@peppapighs
Add Javadoc
68e5d19
angkl0
angkl0 reviewed Sep 1, 2022
View reviewed changes
Copy link

@angkl0 angkl0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, code standard is high. There could be more consistency as to whether there is an empty line before return statement. The ones in tasklist class do not while most in other classes do.

src/main/java/duke/task/Task.java Outdated Show resolved Hide resolved
src/main/java/duke/task/Task.java Outdated Show resolved Hide resolved
yunruu
yunruu reviewed Sep 2, 2022
View reviewed changes
Copy link

@yunruu yunruu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great code, clean and elegant!

peppapighs and others added 3 commits September 4, 2022 13:30
@peppapighs
Increase consistency with blank line before return
ea85b2d 
Some return statements have no blank line before them.

Having blank line before return statements increases consistency in code quality.
@peppapighs
Merge pull request #1 from peppapighs/branch-A-CodeQuality
216e235 
Increase consistency with blank line before return
@peppapighs
Add blank line before return statements
7804190 
There are missing blank lines in TaskList and Duke.

To increase code quality consistency.
peppapighs and others added 17 commits September 4, 2022 13:41
@peppapighs
Add assertion for file creation
9f5440c 
File should be created when interacting with Storage.

Assertion helps with verifying assumption that should be true.

Java assertion is for verifying assumption during runtime and can be turned off.
@peppapighs
Merge pull request #2 from peppapighs/branch-A-Assertions
0852184 
Add assertion for file creation
@peppapighs
Add gradle CI
401c5e2 
Currently no workflow when pushing or merging commits.

CI helps verify commits if they are compilable.

GitHub CI is easy to setup.
Add reminder feature
abfdfa0
Merge branch 'branch-B-Reminders'
0a3b9d9
Add sort feature
d27c760
Merge branch 'branch-C-Sort'
1b7d5c0
Save task list after being sorted
6d14e66
Merge branch 'branch-C-Sort'
ccfe0af
Add notes feature
3136a41
Merge branch 'branch-D-Notes'
29cc699
@peppapighs
Add launcher class for GUI app
188fa5a
@peppapighs
Add user guide for the chatbot
5ec7f02
@peppapighs
Add window title for the GUI app
7d84e9c
@peppapighs
Add product screenshot
d583e91
@peppapighs
Improve code quality from soc-se-script
b3323e9
@peppapighs
Fix build.gradle for cross-platform build
35105a3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yunruu yunruu yunruu left review comments

@angkl0 angkl0 angkl0 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@peppapighs @angkl0 @yunruu