Skip to content
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

Docs guides #236

Merged
merged 4 commits into from
Nov 7, 2021
Merged

Conversation

uosjapuelks
Copy link
Collaborator

No description provided.

@uosjapuelks uosjapuelks added the priority.High Must do! label Nov 7, 2021
@uosjapuelks uosjapuelks added this to the v2.1 milestone Nov 7, 2021
@uosjapuelks uosjapuelks self-assigned this Nov 7, 2021
@@ -58,6 +58,7 @@ Feature | Command Format |
-----------|-----------------
[Add an item](#add-an-item-into-fridget-add) | `add ITEM_NAME /EXPIRY_DATE`
[Remove an item](#remove-an-item-from-fridget-remove) | `remove ITEM_NAME`
[Update the quantity of an item]() | `update ITEM_NAME`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

link is not working yet

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@@ -194,6 +195,19 @@ You have successfully removed:
__________________________________________
```

<ins>If there is only one item with a name containing or with different upper or lower case to `ITEM_NAME`</ins>:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this be changed to "If there is only one item with a similar spelling (different capitalization / has more letters)"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

__________________________________________
```

<ins>If the intended value to be update is Zero</ins>:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo. Should be to "updated is zero"

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

__________________________________________
```

<ins>If there are multiple quantities of the same item</ins>:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this be changed to "If there are multiple items of the same name?"

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deon

@uosjapuelks uosjapuelks merged commit 2443fc4 into AY2122S1-CS2113T-W12-4:master Nov 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants