-
Notifications
You must be signed in to change notification settings - Fork 434
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
[Chuah Jia Jie] iP #473
Open
chuahjiajie
wants to merge
82
commits into
nus-cs2103-AY2324S2:master
Choose a base branch
from
chuahjiajie: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.
Open
[Chuah Jia Jie] iP #473
Conversation
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
Let's tweak the docs/README.md (which is used as the user guide) to fit Duke better. Specifically, 1. mention product name in the title 2. mention adding a product screenshot and a product intro 3. tweak the flow to describe feature-by-feature
# Conflicts: # src/main/java/OedipusRex.java
…uring merge process
…o that bot has GUI
…ious functionality of the chatbot into the form of a GUI
* branch-A-Level-10: Add fully functional GUI that contains images and integrates the previous functionality of the chatbot into the form of a GUI Make standard GUI based on tutorial provided by Week 4 GUI tutorial so that bot has GUI
…e that can be run on any computer
… long, dialog box is no longer cut off
There was an issue with the GUI where the output box would not load more than 5 lines worth of tasks when prompted to list out tasks. Fixing this would allow for a better user experience. As a step towards improving the GUI, let's make sure the code in the MainWindow.fxml file is properly calibrated. Doing so will make the subsequent implementation of GUIs functional.
There were multiple areas in the Parser and other classes that could have benefitted from the use of assertions.
Was going to implement assertions on a separate branch, but I forgot.
* branch-A-Assertions: Add Assertion Branch
Forgot to add level 8 branch lol.
* branch-level-8: Add Level-8 Branch
Made adjustments to code quality
There was an error where the application would display unexpected behaviour when a non positive integer was provided for the delete command.
Changed boolean names in parser.
* branch-A-CodeQuality: Correct Boolean Name Fix Delete Errors Edit Coding Quality
Chatbot now has a help command. User simply needs to type in "help" and is able to see a full list of all the commands for the bot.
* branch-C-Help: Add Help feature
Errors found and corrected in deadline and event classes due to issues with date time feature.
Added picture for Ui.png into docs folder.
Previous test cases no longer valid. Updated with new test cases.
Feature 1 was not added. Adding for fun.
Add Feature 1
Final commit for project.
Added a user guide for Balkan Bot.
Made changes to user guide.
Storage file had methods that were too long.
Parser Method was too long.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
DukePro
DukePro frees your mind of having to remember things you need to do. It's,
FASTSUPER FAST to useAll you need to do is,
And it is FREE!
Features:
If you are a Java programmer, you can use it to practice Java too. Here's the
main
method: