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

NW5-Fatih Celebi- JavaScript-Core-1-Coursework-Week1 - #405

Open
fatih-celebi wants to merge 6 commits into
CodeYourFuture:masterfrom
fatih-celebi:master
Open

NW5-Fatih Celebi- JavaScript-Core-1-Coursework-Week1#405
fatih-celebi wants to merge 6 commits into
CodeYourFuture:masterfrom
fatih-celebi:master

Conversation

@fatih-celebi

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: Fatih
  • Your City: Manchester
  • Your Slack Name: fatih-celebi

Homework Details

  • Module: JS 1
  • Week: 1

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

Comment thread mandatory/4-tax.js
function addTaxAndFormatCurrency() {}

function addTaxAndFormatCurrency(price) {
return (price+(price*20/100)).toFixed(2);

Choose a reason for hiding this comment

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

good expression, you could declare some lets to make the code easier to understand

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.

Thank you for your feedback, Luan.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants