Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

London 10-Anu Thapaliya-js2-week1#249

Open
anuthapaliy wants to merge 4 commits intoCodeYourFuture:mainfrom
anuthapaliy:main
Open

London 10-Anu Thapaliya-js2-week1#249
anuthapaliy wants to merge 4 commits intoCodeYourFuture:mainfrom
anuthapaliy:main

Conversation

@anuthapaliy
Copy link
Copy Markdown

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

@anuthapaliy anuthapaliy added the review requested I would like a mentor to review my PR label Mar 27, 2023
Comment thread 2-mandatory/1-recipes.js
@@ -22,4 +22,38 @@
You should write and log at least 5 recipes
*/

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The recipe objects look good 👍
But take another look at the instructions on lines 13-20. It looks each recipe should be output in this way:

  Mole
  Serves: 2
  Ingredients:
  cinnamon
  cumin
  cocoa


function createLookup(countryCurrencyCodes) {
// write code here
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice 👍


function createShoppingList(recipe) {
// write code here
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This looks good to me 👍
As an extra challenge, can you re-write this using the filter array method?

for (let ingredient of recipe.ingredients)
{

if (pantry.fridgeContents.includes(ingredient)=== false
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

When you see something like this: pantry.fridgeContents.includes(ingredient) === false - you can re-write it like this: !pantry.fridgeContents.includes(ingredient). Can you think of why that works?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

ok great thank you


let cashRegister = {
// write code here
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good 👍

/*
Write a test that checks a score of 55 is grade D
*/

Copy link
Copy Markdown

@moneyinthesky moneyinthesky Mar 31, 2023

Choose a reason for hiding this comment

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

The tests above look good.
But below - it looks like you are missing toEqual?

}
*/

/*
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

These are great, well done!
Good test names as well 😄

@moneyinthesky
Copy link
Copy Markdown

Nice work on this coursework @anuthapaliy!

@moneyinthesky moneyinthesky added the reviewed A mentor has reviewed this code label Mar 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

review requested I would like a mentor to review my PR reviewed A mentor has reviewed this code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants