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

wm3-pezhman-azizi-javascript-week1#192

Closed
Pezhman-Azizi wants to merge 2 commits into
CodeYourFuture:masterfrom
Pezhman-Azizi:master
Closed

wm3-pezhman-azizi-javascript-week1#192
Pezhman-Azizi wants to merge 2 commits into
CodeYourFuture:masterfrom
Pezhman-Azizi:master

Conversation

@Pezhman-Azizi
Copy link
Copy Markdown

@Pezhman-Azizi Pezhman-Azizi commented Aug 11, 2021

I have made all the changes as required in the read me files

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: Pezhman Azizi
  • Your City: West Midlands
  • Your Slack Name: WM3

Homework Details

  • Module:Java Script
  • Week:1

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?


View rendered exercises/D-strings/README.md
View rendered exercises/G-numbers/README.md

I have made all the changes as required in the read me files
All the tests are passed
Copy link
Copy Markdown

@tenzyns tenzyns left a comment

Choose a reason for hiding this comment

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

Review completed.

Comment thread mandatory/4-tax.js

function calculateSalesTax() {}
function calculateSalesTax(price) {
return price * 0.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.

I wrote the same way as you, but I did not pass the test. So I had to calculated total price including the tax in this function calculateSalesTax(). I wrote: return 1.2 * price; and the test then passed.

@github-actions
Copy link
Copy Markdown

Your coursework submission has been closed because nobody has interacted with it in six weeks. You are welcome to re-open it to get more feedback.

@github-actions github-actions Bot added the Stale label Sep 25, 2021
@github-actions github-actions Bot closed this Sep 25, 2021
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.

2 participants