-
Notifications
You must be signed in to change notification settings - Fork 5
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 prefix table, add title to user guide #307
Update prefix table, add title to user guide #307
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.
LGTM
# Conflicts: # docs/UserGuide.md
Codecov Report
@@ Coverage Diff @@
## master #307 +/- ##
=========================================
Coverage 77.14% 77.14%
Complexity 857 857
=========================================
Files 127 127
Lines 2485 2485
Branches 325 325
=========================================
Hits 1917 1917
Misses 477 477
Partials 91 91 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
docs/UserGuide.md
Outdated
--- | ||
|
||
# **1. Introduction** | ||
|
||
# 1. Introduction |
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.
make it bold for consistency
docs/UserGuide.md
Outdated
@@ -303,7 +322,7 @@ Examples: | |||
|
|||
|
|||
|
|||
### 4.1.5. Find employee by ID: `feid` | |||
### 4.1.5. Find employees by ID: `feid` |
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 be "employee" because only one employee has the ID
docs/UserGuide.md
Outdated
|
||
--- | ||
* Table of Contents | ||
{:toc} |
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.
remove indent
docs/UserGuide.md
Outdated
|-------------------------|----------------------|-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------| | ||
| **ID** | id/ | {::nomarkdown} ID is the unique identifier for an employee. Leading zeroes are ignored. <br/> **Note: This flag is only used when adding an employee into the app**. <br/> It has the following constraints: <br/> <ul> <li> Must be a positive integer <ol> <li> 0 is often reserved for administrative use </li> <li> negative numbers are not conventionally used </li> </ol> </li> <li> Cannot be empty </li> </ul> <br/> Valid Examples: <ul> <li>1</li> <li>100</li> </ul> Invalid Examples: <ul> <li>Bob</li> <li>0</li> <li>-1</li> </ul> {:/} | | ||
| **ID** | id/ | {::nomarkdown} ID is the unique identifier for an employee. Leading zeroes are ignored. <br/> **Note**: This flag is only used when adding an employee into the app. <br/> It has the following constraints: <br/> <ul> <li> Must be a positive integer <ol> <li> 0 is often reserved for administrative use </li> <li> negative numbers are not conventionally used </li> </ol> </li> <li> Cannot be empty </li> </ul> <br/> Valid Examples: <ul> <li>1</li> <li>100</li> <li>001000</li> </ul> Invalid Examples: <ul> <li>Bob</li> <li>0</li> <li>-1</li> </ul> | | ||
| **EMPLOYEE_ID** | eid/ | Similar to ID, The EMPLOYEE_ID represents the ID of an employee. <br/> It is used for any commands that require referencing an employee. <br/> It follows the same constraints as ID. {:/} | |
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.
need to remove
# Conflicts: # docs/UserGuide.md
No description provided.