Update Use Case #45
Merged
Update Use Case #45
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
underthehai
requested review from
jeannetoh99,
lyeyixian,
jiaweiteo and
timjkong
jiaweiteo
approved these changes
Sep 25, 2020
docs/DeveloperGuide.md
Outdated
| * **Person**: Contains information about a given person `name`, `mobile number` and `passport number` | ||
| * **Directory**: The three possible directories are `top`, `travelplan`, `wishlist` | ||
| * **Navigate**: Use the `goto` command to move between directories | ||
| * **View**: A command that provides a list of specified items |
There was a problem hiding this comment.
Think the command is called 'show' instead?
There was a problem hiding this comment.
yeap, I will change it to show
lyeyixian
reviewed
Sep 26, 2020
|
|
||
| **Extensions** | ||
|
|
||
| * 1a. User is at the top directory. |
There was a problem hiding this comment.
I'm a bit not sure about this. Is it we can edit the travel plan at top directory also?
There was a problem hiding this comment.
I was thinking that since we can view the travel plans at the top directory, we can also modify the start and end date there too. Or is it better to restrict the editing to within travel plan directory?
There was a problem hiding this comment.
There was a problem hiding this comment.
i think this is ok but we should have a command to edit start and end date within travel plan directory also if not it might be a bit confusing
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.
Add UC11 to UC17 and glossary to DeveloperGuide.md. Edited some parts of edit command under UserGuide.md.
close #41 #43