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

londonclass8 -Laleh Koupaei-JavaScript-Week1 - #231

Closed
lalehkoupa wants to merge 2 commits into
CodeYourFuture:masterfrom
lalehkoupa:master
Closed

londonclass8 -Laleh Koupaei-JavaScript-Week1#231
lalehkoupa wants to merge 2 commits into
CodeYourFuture:masterfrom
lalehkoupa:master

Conversation

@lalehkoupa

@lalehkoupa lalehkoupa commented Dec 13, 2021

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:Laleh
  • Your City:London
  • Your Slack Name:Lalehkoupa

Homework Details

  • Module:Javascript
  • Week:1

Notes

  • What did you find easy?writing Javascript code

  • What did you find hard?Testing

  • What do you still not understand?the concept of npm test and package.json file

  • Any other notes?


View rendered exercises/B-hello-world/README.md

Comment thread mandatory/4-tax.js
function addTaxAndFormatCurrency() {}
function addTaxAndFormatCurrency(itemPrice) {
let totalPrice = calculateSalesTax(itemPrice);
let formatPrice = "£" + totalPrice.toFixed(2);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi Laleh, good job with the tax homework. You can also use string interpolation in your formatPrice as another option. Well done!!

@sobiawaqar sobiawaqar left a comment

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 Work Laleh!!

@Simon2828 Simon2828 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Great effort, Laleh - keep it up! 😁

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants