London | 26-ITP-Sep | Anita Amirhaeri | Sprint 3 | coursework/sprint 3 by anitahy73 · Pull Request #1636 · CodeYourFuture/Module-JavaScript-Fundamentals

anitahy73 · 2026-10-01T19:45:29Z

Learners, PR Template

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

oursework/sprint 3

Questions

Added comments to explain variable declaration and assignment.

netlify · 2026-10-01T19:45:36Z

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	`64e95e7`
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6acbc6862debd40008ca0a67
😎 Deploy Preview	https://deploy-preview-1636--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.

Removed placeholder comments and explanations in time-format.js.

Added comments to explain the behavior of the pad function and its calls in formatTimeDisplay.

abdishakoor-dev · 2026-10-06T21:11:51Z

Before I can review your Sprint 3 work, this PR needs to change Sprint 3 files only. At the moment it also changes 5 Sprint 2 files, which is why the bot keeps taking the Needs Review label off:

Sprint-2/1-key-exercises/1-count.js
Sprint-2/1-key-exercises/3-paths.js
Sprint-2/2-mandatory-errors/4.js
Sprint-2/3-mandatory-interpret/1-percentage-change.js
Sprint-2/3-mandatory-interpret/3-to-pounds.js

Your Sprint 2 work is already completed. It's in your Sprint 2 PR (#1553), which is already marked Complete. So we can simply remove the Sprint 2 files from this PR:

Open the project in VS Code. In the bottom-left corner, check the branch name says acoursework/sprint-3. If it doesn't, click it and pick acoursework/sprint-3 from the list.
Open a terminal: Terminal menu, then New Terminal.
Copy and paste these lines one at a time, pressing Enter after each:

git pull
git fetch https://github.com/CodeYourFuture/Module-JavaScript-Fundamentals.git main
git checkout FETCH_HEAD -- Sprint-2
git commit -m "Remove Sprint-2 changes from this PR"
git push

Refresh this page and click the Files changed tab at the top. You should see only Sprint-3 files (10 of them), and no Sprint-2 files.
Add the Needs Review label again. This time it will stay.

If anything looks different from what I've described, or you get an error, post a screenshot here and I'll help you through it.

Removed duplicate comment and fixed typo.

abdishakoor-dev

Thanks for clearing out the Sprint 2 files. Your answers are right almost everywhere, however, there are there are still few issues to resolve before I can mark it Complete:

Four files stop with an error before they reach your fix: 1-key-errors/0.js, 1.js, 2.js and 2-mandatory-debug/2.js. Put // in front of the original code in each, run node on the file, and check your new code prints what you expect.
1-key-errors/1.js line 25 has a stray ca.
2-mandatory-debug/2.js line 30 declares num a second time.
time-format.js b): check which value pad gets first (line 36).
1-bmi.js: call calculateBMI with an example and print the result.
Format Document on 1-key-errors/1.js and 2.js and save.

Add the Needs Review label again when you've pushed.

abdishakoor-dev · 2026-10-12T11:52:55Z

+  str = `${str[0].toUpperCase()}${str.slice(1)}`;
+  return str;
+}
+console.log(capitalise("mother"));


Run node 0.js: node stops at line 14 with the SyntaxError, so this line never runs. Put // in front of lines 13 to 16, the original code. Lines 6 to 10 and 20 to 22 do the same thing now, so you only need one of them.

abdishakoor-dev · 2026-10-12T11:52:55Z

 // =============> write your new code here
+function convertToPercentage(decimalNumber) {
+  const percentage = `${decimalNumber * 100}%`;
+ ca


What is ca doing here? Once lines 10 to 17 are in comments, run the file and see what node says about it.

abdishakoor-dev · 2026-10-12T11:52:55Z

-// =============> write your prediction of the error here
+// =============> write your prediction of the error here:We will have a SyntaxError because we need a valid function parameter. 

 function square(3) {


Your explanation is right. Now put // in front of lines 8 to 10, so that node can run your fixed square on line 20. This also lets Format Document work on the file.

abdishakoor-dev · 2026-10-12T11:52:55Z

 // Finally, correct the code to fix the problem
-// =============> write your new code here
+// =============> write your new code here:
+const num = 103;


node stops here: SyntaxError: Identifier 'num' has already been declared, because line 6 already made num. Put the original code (lines 6 to 14) in comments. Then does your new getLastDigit need this line at all?

abdishakoor-dev · 2026-10-12T11:52:55Z

+//totalMinutes = (61 - 1) / 60 → 1
+//remainingMinutes = 1 % 60 → 1
+//totalHours = (1 - 1) / 60 → 0
+//num = 1


Line 15 calls pad(totalHours) first. On line 35 you worked out totalHours is 0, and your answer to c) starts from 0 too. So what is num the first time?

abdishakoor-dev · 2026-10-12T11:52:55Z

 function calculateBMI(weight, height) {
-  // return the BMI of someone based off their weight and height
+  const bmi = weight / (height * height);
+  return bmi.toFixed(1);


toFixed(1) gives the string the spec asks for. Good. Call it with the example from line 5 and print the result: do you get "23.4"? Your other two implement files already do this.

anitahy73 added 12 commits September 20, 2026 19:53

Add comments for count variable and assignment

2300ec7

Added comments to explain variable declaration and assignment.

Fix capitalise function variable redeclaration

a9cb566

Fix capitalise function variable redeclaration

a729eff

fix percentage conversion variable redeclaration

7cd8ae3

Fix square function parameter

a128bbf

Fix multiply function to return result

1d044cb

Fix sum function return statement

d1a35bc

Fix getLastDigit function parameter

35c7d91

Implement BMI calculation function

2d78dfc

Implement UPPER_SNAKE_CASE conversion function

6956732

converting the function into reusable toPounds function

76ac665

writing the answers of formatTimeDisplay function

0b2d01c