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

Update ug #178

Merged
merged 15 commits into from
Oct 26, 2020
Merged

Conversation

caitlinjee
Copy link
Collaborator

No description provided.

@codecov-io
Copy link

codecov-io commented Oct 25, 2020

Codecov Report

Merging #178 into master will decrease coverage by 0.21%.
The diff coverage is 8.33%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #178      +/-   ##
============================================
- Coverage     64.54%   64.32%   -0.22%     
  Complexity      685      685              
============================================
  Files           116      116              
  Lines          2383     2391       +8     
  Branches        294      294              
============================================
  Hits           1538     1538              
- Misses          733      741       +8     
  Partials        112      112              
Impacted Files Coverage Δ Complexity Δ
...seedu/address/logic/commands/AddRecipeCommand.java 100.00% <ø> (ø) 8.00 <0.00> (ø)
...va/seedu/address/logic/commands/CommandResult.java 70.90% <ø> (ø) 18.00 <0.00> (ø)
...seedu/address/logic/commands/EatRecipeCommand.java 89.47% <ø> (ø) 9.00 <0.00> (ø)
.../address/logic/commands/EditIngredientCommand.java 83.67% <ø> (ø) 13.00 <0.00> (ø)
...eedu/address/logic/commands/EditRecipeCommand.java 92.40% <ø> (ø) 13.00 <0.00> (ø)
...du/address/logic/commands/SearchRecipeCommand.java 100.00% <ø> (ø) 6.00 <0.00> (ø)
...du/address/logic/commands/SelectRecipeCommand.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
src/main/java/seedu/address/ui/MainWindow.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...u/address/logic/commands/AddIngredientCommand.java 92.30% <100.00%> (ø) 14.00 <1.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4dff64a...486206c. Read the comment docs.

e.g. in `addR`, `add` is in small letters while `R` is in capital letters.

* Words in `UPPER_CASE` are the parameters to be supplied by you.<br>
e.g. in `addR n/NAME`, `NAME` is a parameter which can be used as `addR n/salad`.

* Items in square brackets are optional.<br>
e.g `n/NAME [t/TAG]` can be used as `n/Salad t/healthy` or as `n/Salad`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Items with …​ after them can be used more than 1 times.
e.g. [t/TAG]…​ can be used as t/healthy, t/healthy t/low calories etc.

Not sure why this is not shown here but can change the ',' after t/healthy to 'or' so its clearer? and remove the etc to make it consistent

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ok good idea

Comment on lines 127 to 130
* Note: If you want to use an image that has been saved in your computer, you have to specify the **absolute path** of the image, and add file:// in front of the file path. Then, replace IMAGE in img/IMAGE with the file path of your image, e.g. img/file:///D:/myimages/wishful/data/myimage.png.
* Note: If you input an invalid file path, Wishful Shrinking will replace the image with a default image.
* Note: If you want to use a url as the image, make sure that you are connected to the internet on your computer, and that the url is the **image address** (starting with https:// and ending with .jpg/jpeg/png). The image will be downloaded into Wishful Shrinking's data folder if there is internet connection.
* Note: If you input an invalid url, or input a url while there is no internet, Wishful Shrinking cannot add the image to the recipe. You have to connect to the internet, or check that your url is a valid image address.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we include a table of built in image paths, e.g. images/healthy1.jpg together with their respective descriptions, just in case users want to use our sample images?

Copy link
Collaborator

Choose a reason for hiding this comment

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

for the last Note, if input invalid url/valid url without internet, Wishful Shrinking will use default image instead.

@@ -26,26 +26,36 @@ Choose a topic from the table of contents to get started on your Wishful Shrinki

1. Ensure your computer has Java `11` or above installed.

2. Download the latest `wishfulShrinking.jar` from [here](https://github.com/AY2021S1-CS2103T-W10-2/tp).
2. Download the latest `wishfulShrinking.jar` from [here](https://github.com/AY2021S1-CS2103T-W10-2/tp/releases).

3. Copy the file to an empty folder you want to use as the _home folder_.

4. **Double-click** the jar file to start the app OR start the app using **CLI** and type `java -jar <jar file name>.jar`.<br>
Copy link
Collaborator

Choose a reason for hiding this comment

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

should we specify our jar file name as wishfulShrinking.jar?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

what do you mean by this?

* Command Result: The Command Result box will show the result of your input into the Command Box.
* Command Box: Here is where you will type all your commands.

6. Type the command in the command box and press `Enter` to execute it. e.g. typing **`help`** and pressing Enter will open the help window.<br>
Copy link
Collaborator

Choose a reason for hiding this comment

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

can remove the eg help since the examples are all already below?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think this eg is ok since it specifies that the user has to press enter also? below it doesnt specify the enter


* This command closes the left drawer that is opened after you have selected a recipe.
* Note: Nothing will happen if you use this command when the left drawer is not opened.
<br><br><br>

### Deleting a recipe : `deleteR`
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can add on explanation that deleteR will not affect the consumption list.

e.g. in `addR`, `add` is in small letters while `R` is in capital letters.

* Words in `UPPER_CASE` are the parameters to be supplied by you.<br>
e.g. in `addR n/NAME`, `NAME` is a parameter which can be used as `addR n/salad`.

* Items in square brackets are optional.<br>
e.g `n/NAME [t/TAG]` can be used as `n/Salad t/healthy` or as `n/Salad`.

Copy link
Collaborator

Choose a reason for hiding this comment

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

The image above is a labeled diagram of each of Wishful Shrinking's components. Below is a brief explanation on each of the components.

Should separate the image title from the second part. (Image title/descr should be in own line)

* there is a mandatory space before `-`
Format: `addR n/NAME i/INGREDIENT [ -QUANTITY][, MORE INGREDIENTS [ -QUANTITY]] c/CALORIES img/IMAGE inst/INSTRUCTION... [t/TAG]...`

* `INGREDIENT` can take in an optional `Quantity` e.g. i/Tomato -2 whole.
Copy link
Collaborator

Choose a reason for hiding this comment

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

QUANTITY only accepts alphanumeric characters, forward slashes and full stops.

* Note: If you want to use an image that has been saved in your computer, you have to specify the **absolute path** of the image, and add file:// in front of the file path. Then, replace IMAGE in img/IMAGE with the file path of your image, e.g. img/file:///D:/myimages/wishful/data/myimage.png.
* Note: If you input an invalid file path, Wishful Shrinking will replace the image with a default image.
* Note: If you want to use a url as the image, make sure that you are connected to the internet on your computer, and that the url is the **image address** (starting with https:// and ending with .jpg/jpeg/png). The image will be downloaded into Wishful Shrinking's data folder if there is internet connection.
* Note: If you input an invalid url, or input a url while there is no internet, Wishful Shrinking cannot add the image to the recipe. You have to connect to the internet, or check that your url is a valid image address.
* Each `INSTRUCTION` is separated by a `.` e.g. instr/Cook. Eat.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Each INSTRUCTION could be confusing bc there is only 1 instr tag

INSTRUCTION will take in a series of instruction text and Wishful Shrinking will automatically separate each step of the instruction based on the end of a sentence, indicated by a .. e.g. instr/Cook pasta. Serve immediately.

* Note: The index **must be a positive integer** e.g. 1, 2, 3......

Examples:
* `selectR 1` shows the 1st recipe in full view (left drawer), as well as in the command result box on the right.
Copy link
Collaborator

Choose a reason for hiding this comment

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

not sure if necessary to write this "command result box on the right." since the messages in the command result box is true for all commands.

![help message](images/helpMessage.png)
<br>The image above is Wishful Shrinking's help window.<br><br><br>
Copy link
Collaborator

Choose a reason for hiding this comment

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

place image title/descr before image

* Note: The index **must be a positive integer** e.g. 1, 2, 3......
* Note: All fields are optional, but **at least one** of the fields must be present. There can be any number of fields to edit.
* Note: You are not allowed to edit a recipe into another recipe that is already in the recipe list.
* Note: Use carefully as this command will override the existing values with the new values.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Note: Specified fields will permanently override existing values with the new values. To directly modify existing recipe ... link to the tip below: Tip: Typing editR INDEX and then hitting Enter will insert the information of the recipe at the specified INDEX in...

Examples:
* `recommend` returns `salad` with ingredients `lettuce`, `onion` and `tomato` only if the user has all `lettuce`, `onion` and `tomato` in the fridge
* `recommend` returns the recipe `salad` with ingredients `lettuce`, `onion` and `tomato` **only if** you have all `lettuce`, `onion` and `tomato` in your fridge.
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • recommend returns the recipe salad with ingredients lettuce, onion and tomato since the example user has all the ingredients: lettuce, onion and tomato in their fridge.


Format: `recommend`

* Note: Recipes will only be recommend if you have all the ingredients in the recipe with the exact same name in your fridge.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Note: Ingredient quantity is not taken into account when determining if an ingredient is present or absent in your fridge.


Format: `addF i/INGREDIENT [ -QUANTITY][, MORE INGREDIENTS [ -QUANTITY]]`

* `INGREDIENT` can take in an optional `Quantity` e.g. i/Tomato -2 whole.
Copy link
Collaborator

Choose a reason for hiding this comment

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

QUANTITY only accepts alphanumeric characters, forward slashes and full stops.

* Note: Use carefully as this command will override the existing values with the new values
* Tip: Use `editF INDEX` to directly modify the existing ingredient
* Note: The index refers to the index number shown in the displayed ingredient list.
* Note: The index **must be a positive integer** e.g. 1, 2, 3......
Copy link
Collaborator

Choose a reason for hiding this comment

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

same as above edit

@jiaqi20 jiaqi20 merged commit b1d60d8 into AY2021S1-CS2103T-W10-2:master Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants