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
Update docs/UserGuide.adoc #89
Update docs/UserGuide.adoc #89
Conversation
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.
Good job, just need to change a bit of stuffs will do
…lication's objectives
- Merge conflicts in AddCommandSystemTest resolved.
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.
Good update, just need to change a few stuffs before merging
docs/UserGuide.adoc
Outdated
• *Reset the application*: `reset` | ||
• *Exit application*: `exit` | ||
//• *Restrict access to BankAB*: `restrict` |
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.
The comment can be deleted since we will not be using it.
public void parseCommand_undoCommandWord_returnsUndoCommand() throws Exception { | ||
assertTrue(parser.parseCommand(UndoCommand.COMMAND_WORD) instanceof UndoCommand); | ||
assertTrue(parser.parseCommand("undo 3") instanceof UndoCommand); | ||
} | ||
}*/ |
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.
The comment can be deleted since we will not be using it.
…and Redo commands, which we have removed
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.
PR approved.
Removal of commands
Miscellaneous