-
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
Update command summary and add internship use cases #122
Update command summary and add internship use cases #122
Conversation
docs/DeveloperGuide.md
Outdated
Guarantees: Addition of internship is successful | ||
|
||
1. User requests to add an internship to a company and provides the relevant details. | ||
2. InternHunter adds the internship to the list of internships of the company. |
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 think you may be missing a line break here?
docs/UserGuide.md
Outdated
**Application**`| **Delete** | `delete app INDEX` | ||
| **Edit** | `edit app INDEX [s/STATUS] [d/STATUS_DATE]` | ||
| **View** | `view app INDEX` | ||
**Profile** | **Add** | `add me c/CAT d/DESCRIPTORS` |
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 profile command is updated with [t/TITLE]
and [d/DESCRIPTORS]...
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.
LGTM, good work! Some merge conflicts to resolve first.
Fixed! |
#64 #115