-
Notifications
You must be signed in to change notification settings - Fork 6
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
Dennis branch #100
Dennis branch #100
Conversation
…r. Change App Icon in MainWindow.fxml
…r. Change App Icon in MainWindow.fxml
…r. Change App Icon in MainWindow.fxml
…r. Change App Icon in MainWindow.fxml
…r. Change App Icon in MainWindow.fxml
…r. Change App Icon in MainWindow.fxml
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.
Command works. The help window appeared when I typed in help command. I think can consider adding the Help command also within the help window since it is technically a command. Approved. Formatting and design I think can be improved 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.
Help command works. Gui can be improved later.
"format: \nadd n/<name> s/<source of stock> q/<quantity> l/<location in warehouse>\n\n"; | ||
|
||
public static final String DELETE_DESCRIPTION = | ||
"format: \ndelete sn/<serial number>\n\n"; |
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.
Should we include an example which shows the usage of deleting multiple stock?
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.
Yes, I think it would be good to include deleting multiple stock at once.
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.
I agree!
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.
Noted I will include them into the usage!
|
||
public static final String UPDATE_DESCRIPTION = | ||
"format(multiple): \nupdate sn/<Serial Number of product> (followed by one of):\n" | ||
+ "q/<+/-><quantity to increment/decrement> nq/<new quantity> n/<new name> " |
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.
Based on our last meeting, I think prefix for quantity to increment/decrement should be changed to "iq/"?
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.
Noted! I will include the prefix for the quantity! I will make the necessary changes on the UG as well
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.
Will be good to merge after those minor changes!
No description provided.