London | 26-ITP-SEPT | Carol Nassuna | Sprint 3 | Coursework by Mugs3 · Pull Request #1606 · CodeYourFuture/Module-JavaScript-Fundamentals

Mugs3 · 2026-09-27T09:38:52Z

Self checklist

I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
My changes meet the requirements of the task
I have tested my changes
My changes follow the style guide

Task code

CYF-1053

Changelist

30/09 - Corrected issues from run 1

I have updated all Sprint-3 files in the key errors, mandatory debug, mandatory implement and mandatory interpret folders

…ory implement and mandatory interpret folders

netlify · 2026-09-27T09:38:58Z

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	`3365fc9`
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6abd7dd4432b5d0008b98565
😎 Deploy Preview	https://deploy-preview-1606--cyf-onboarding-module.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	2 paths audited Performance: 100 (no change from production) Accessibility: 100 (no change from production) Best Practices: 100 (no change from production) SEO: 86 (no change from production) PWA: - View the detailed breakdown and full score reports
🤖 Make changes	Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

hey-hammad · 2026-09-29T12:33:50Z

+console.log(capitalise("beautiful"));

-// =============> write your explanation here
+// function is assigned 2 arguments/values=============> write your explanation here


Please review the error message again and update the explanation. Think about what the error message is about and why it needs to be fixed.

hey-hammad · 2026-09-29T12:46:26Z

 console.log(decimalNumber);

-// =============> write your explanation here
+// decimalNumber has been declared as a function and a variable within function scope=============> write your explanation here


decimalNumber is not a function.

Please review the explanation again and think about the following question and finally update the explanation.

how is a function declaration different than a variable?

what are function arguments and parameters?

can a function variable and parameter have the same name?

hey-hammad · 2026-09-29T12:51:17Z

+// Syntax errorError Unexpected number.=============> write the error message here

-// =============> explain this error message here
+// Means Function identifier should be a name not a number =============> explain this error message here.


Whats the difference between function identifier and parameter?

I think you're confused between the function identifiers, parameters and arguments.

please review the following and update the explanation.

https://medium.com/@kumarsmt88/identifiers-in-javascript-275a955cbd4a

https://www.w3schools.com/js/js_function_arguments.asp

https://www.w3schools.com/js/js_function_parameters.asp

hey-hammad · 2026-09-29T12:53:47Z


 // =============> write your explanation here
-
+//The 2nd console log returned value value is undefined. I have added a return function to output the multiply value.


why was the value returned by the function is undefined?

hey-hammad · 2026-09-29T12:58:01Z

 console.log(`The sum of 10 and 32 is ${sum(10, 32)}`);

-// =============> write your explanation here
+// Maybe that unassigned return; may cause the code not to run fully =============> write your explanation here


When you don't understand how or why a particular part of the code works or behaves in a certain way, it's best to dig a little deeper so you can build a solid understanding of it.

Please review the following MDN documentation on the return statement to help you understand how function returns work:

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/return

hey-hammad · 2026-09-29T13:02:37Z

  // return the BMI of someone based off their weight and height
 }
+
+function calculateBMI(weight, height) {


pleaser review the formula for calculating the BMI in the description above again.

hey-hammad · 2026-09-29T13:07:40Z

 // This might help https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/toUpperCase
+
+function toUpperSnakeCase(text) {
+  return text.trim().replace(/\s+/g, "_").toUpperCase();


is there a need to use trim and regex pattern here? is there an easy of achieving this without using them?

hey-hammad · 2026-09-29T13:10:04Z

+console.log(`£${pounds}.${pence}`);*/
+
+// Revised code:
+function poundsToPence(penceString) {


function name is a bit misleading.

hey-hammad · 2026-09-29T13:15:57Z


 // c) What is the return value of pad when it is called for the first time?
 // =============> write your answer here
+0;


please review the answer again.

hey-hammad · 2026-10-02T10:30:18Z

+// Syntax errorError Unexpected number.=============> write the error message here

-// =============> explain this error message here
+// Means Function identifier should be a name not a number =============> explain this error message here.


I think you're confused between the function identifiers, parameters and arguments.

please review the following and update the explanation.

https://medium.com/@kumarsmt88/identifiers-in-javascript-275a955cbd4a

https://www.w3schools.com/js/js_function_arguments.asp

https://www.w3schools.com/js/js_function_parameters.asp

hey-hammad · 2026-10-02T10:38:41Z

+
+function calculateBMI(weight, height) {
+  // return (weight / height).toFixed(1); - 1st attempt
+  return (weight / (height * height)).toString().slice(0, 4);


thats the correct formula now, however do you need to use .toString() and .slice() for getting fixed decimal points? is there an easier way of doing it without converting the value to string?

hey-hammad · 2026-10-02T10:49:18Z

+console.log(text);
+
+// Trim and regex pattern was the only saving grace I came across to resolve the underscore issue I had spent hours researching.
+// I have however now modified and replaced the pattern for easy understanding with AI's help


I appreciate your transparency. Using AI as a tool for research and to help clarify concepts you're struggling with is absolutely fine. You can find further guidance on the appropriate use of AI in the document below.

https://docs.google.com/document/d/1Rq3dh3yE7sqkBM1cTPP6-f2CSO4lhdhi_2yXxAV3EfA/edit?tab=t.0#heading=h.3kept1c7ccss

Mugs3 added 2 commits September 25, 2026 19:03

updated files in key errors and mandatory-debug folders

f97b348

updated all Sprint-3 files in the key errors, mandatory debug, mandat…

620e7c7

…ory implement and mandatory interpret folders

Mugs3 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 27, 2026

hey-hammad added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Sep 29, 2026

hey-hammad suggested changes Sep 29, 2026

View reviewed changes

hey-hammad added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Sep 29, 2026

corrected issues raised from run 1

3365fc9

Mugs3 added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. and removed Reviewed Volunteer to add when completing a review with trainee action still to take. labels Sep 30, 2026

hey-hammad added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Oct 2, 2026

hey-hammad suggested changes Oct 2, 2026

View reviewed changes

hey-hammad added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Oct 2, 2026


		// =============> write your explanation here

		//The 2nd console log returned value value is undefined. I have added a return function to output the multiply value.

Uh oh!

Conversation

Mugs3 commented Sep 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Task code

Changelist

Uh oh!

netlify Bot commented Sep 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for cyf-onboarding-module ready!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Mugs3 commented Sep 27, 2026 •

edited

Loading

netlify Bot commented Sep 27, 2026 •

edited

Loading