mandatory part update

[fowobi]

finish the mandatory part

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

Glasgow Class 6 - Tosin Odunuga - JS1 - Week 1

Your Details

• Your Name:Tosin_Odunuga
• Your City: Glasgow class 6
• Your Slack Name: Tosin

Homework Details

• Module:JavaScript
• Week: 1

Notes

• What did you find easy?

• What did you find hard?

• What do you still not understand?

• Any other notes?

[gregdyke]

Hi Tosin

This is good work, well done!

Next time you open a pull request could you give it the title we request? Something like

Glasgow Class 6 - Tosin Odunuga - JS1 Week 1

[gregdyke]

Can you add a comment to explain what this function does (instead of changing the function?)

[fowobi]

Okay I will do that

[gregdyke]

nicely done!

[fowobi]

Ok,I will correct it. Thank you

…

On Wed, 22 Feb 2023, 13:17 gregdyke, ***@***.***> wrote: ***@***.**** commented on this pull request. Hi Tosin This is good work, well done! Next time you open a pull request could you give it the title we request? Something like Glasgow Class 6 - Tosin Odunuga - JS1 Week 1 ------------------------------ In mandatory/3-function-output.js <#507 (comment)> : > function getRandomNumber() { return Math.random() * 10; } // Add comments to explain what this function does. You're meant to use Google! function combine2Words(word1, word2) { - return word1.concat(word2); + Can you add a comment to explain what this function does (instead of changing the function?) ------------------------------ In mandatory/4-tax.js <#507 (comment)> : > @@ -17,7 +19,10 @@ function calculateSalesTax() {} Remember that the prices must include the sales tax (hint: you already wrote a function for this!) */ -function addTaxAndFormatCurrency() {} +function addTaxAndFormatCurrency(price) { + let pricewithtax = calculateSalesTax(price); + return '£' + pricewithtax.toFixed(2); nicely done! — Reply to this email directly, view it on GitHub <#507 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A2GGUINFCH6YDXVLZC4HPFLWYYGV5ANCNFSM6AAAAAAVCIZLGE> . You are receiving this because you authored the thread.Message ID: <CodeYourFuture/JavaScript-Core-1-Coursework-Week1/pull/507/review/1309207137 @github.com>