Manchester | 26-ITP-Sep | Mathew Shaibu | Sprint 2 | Course-work-2 by MattCarolus10 · Pull Request #1546 · CodeYourFuture/Module-JavaScript-Fundamentals

MattCarolus10 · 2026-09-19T20:37:25Z

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-1039

Changelist

I have completed the "1-key-exercises" and corrected the syntax errors.
I checked for errors in the "2-Mandatory-errors" task and completed it.
I interpreted every lines of code in the "3-Mandatory-interpret" task and completed it.
I used the DevOps tool to check and research some JavaScript functions on chrome

Questions

Most functions in this project are new to me, even though I have been able to understand their functions individually, I am not really confident to code with them. I will keep practicing and any advise would really be appreciated.

netlify · 2026-09-19T20:37:29Z

✅ Deploy Preview for cyf-onboarding-module ready!

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

hackertainment · 2026-09-24T18:00:14Z

You don't need to submit the files under the Prep folder. Therefore, I didn't read them.
Generally, there are still rooms to improve the explanation by:

adding a bit more key details
using correct words to describe
re-organising the order of presentation

Good try, and keep it up.

MattCarolus10 · 2026-09-24T18:06:42Z

Thank you very much

hackertainment · 2026-09-24T17:24:16Z

+const dir = filePath.slice(0, lastSlashIndex);
+console.log(`The dir part of ${filePath} is ${dir}`);
+
+const ext = base.slice(4);


The file name does not necessary 4 characters long. It can be longer and shorter (e.g. considering /path/to/screenshot.png). Cutting out the first 4 characters does not necessary remain the file extension. Please fix it, thank you.

Thank you for this review, I have checked and found out the right solution. Appreciate

hackertainment · 2026-09-24T17:33:15Z

+// num variable that carries the result of the expressions evaluated.
+// (maximum - minimum) does basic math: 100 - 1 + 1 = 100
+// Math.floor() takes out all decimal and make them an integer.
+// Math.random() gives random numbers between 0 and 100.
+// + minimum add 1 at the end but randomise to the value between 0 and 100


Math.random() itself does not give a number between 0 to 100.

+ minimum would not randomise value.

The explanation require a bit more details and clarity.

It would be better for your breakdown to following the same order of expression evaluation.

Please fix and elaborate you answer, thank you.

Thank you, review acknowledged

hackertainment · 2026-09-24T17:35:51Z

+const HourClockTime1 = "8:53pm";
+const hourClockTime2 = "20:53";
+console.log(`${HourClockTime1}, ${hourClockTime2}`)


While this fixed the error, can you think of better variable names? Thank you.

Variable names changed, thank you

hackertainment · 2026-09-24T17:38:36Z

+
 // e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
+
+// Answer: ReplaceAll() function clears out all the commas in the strings, the number() converts the strings to numbers to help with calculation


JavaScript is case sensitive. It should be "The Number() converts ..."

Thank. you, Error corrcted.

hackertainment · 2026-09-24T17:40:33Z

+
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
+
+// Answer: It will work for integer values and will give us clear and clean result using this code. However, the result would not be clean as this using float values


One more situation that make this program won't work well even an integer value is provided. Can you think of it?

I will check and make research to see

hackertainment · 2026-09-24T17:47:28Z

+// Also, line 3-6 used the function substring(0, penceString.lrength - 1) to cut out the "p" in the "penceString" variable. 
+// By starting count fron 0 (beginnning) of "399p".
+// "Using penceString.lenght - 1" to check the length of the value in the variable penceString = "399p" (4),
+//  And deducting 1 from the value (4 - 1), because length - 1 subtract 1 from the character/value count 4. leaving the value at "399"


While the result of getting "399" is correct, the explanation seems not quite accurate to me.

it is not counting, it should be indexing

the reason for deducting 1 is due to how the second parameter of substring() works

Please revise your answer. Thank you.

My explanation is due to my level of understanding, I need to work on my terminologies usage going forward. Thank you

hackertainment · 2026-09-24T17:50:27Z

+//  And deducting 1 from the value (4 - 1), because length - 1 subtract 1 from the character/value count 4. leaving the value at "399"
+
+// Line 8 a variable "paddedPenceNumberString" was created and a function padStart(3, "0") 
+// was used to ensure the value of the variable remains at 3 and to be filled with "0" at the beginning if the value is less 3


Your understanding might be correct but the presentation seems not quite right. What does the "value" in your explanation "the value of the variable remains at 3" mean? Can you revise the wording a bit please? Thank you.

p.s.: same issue with line 50-51 below, and please revise it as well.

Thank you so much for all your reviews, I really appreciate them and I have taken notes of the corrections.

MattCarolus10 added 29 commits September 13, 2026 12:51

Hello_World JavaScript file created

919f2bc

restore password.js to prep

f93c281

1-count.js completed

9f085a5

initials.js task completed

0194cb3

file path task completed

6c5a8c8

4-random task completed

b1dcb1c

0.js error fixed

aed6985

add ; to lines

dd027fa

add ; to lines

3e2a1d2

add ; to lines

b1660fd

task 1.js completed

ed261ff

task 2.js completed

5031f8f

3.js task completed

ddc8ba6

task 4.js completed

8c577d5

percentage-change.js task completed

e44ee66

completed task 2-time-format

48b94e2

explained task 0.js

509ff84

task 1.js explained

f80c15b

task 2.js explained

7d0d970

task 3.js explained and completed

5b4ed9d

task 4.js completed and explained

c41aa67

completed and printed task 1-count.js

08af9b8

task 1-percentage-change explained

536761c

task 2-time-format.js explained and completed

276f457

3-to-pounds

4aa0e25

interpreted task 3-to-pounds.js

4cc01f5

fix: correct console log messages for base and dir parts in 3-paths.js

0a50eb5

fix: update console log for ext part in 3-paths.js

ae41a8e

task 1-kkey-exercise completed

a896a16

MattCarolus10 added 4 commits September 22, 2026 20:57

task 1.js completed

79db8a6

task 2.js completed

fb93c79

clear spaces

fb14e2c

task mandatory-debug completed

5839e8d

hackertainment 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 24, 2026

hackertainment mentioned this pull request Sep 24, 2026

Get Code Review and Iterate hackertainment/My-Coursework-Planner#8

Open

hackertainment 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 24, 2026

hackertainment reviewed Sep 25, 2026

View reviewed changes

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

This comment has been minimized.

Sign in to view

github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 25, 2026

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

This comment has been minimized.

Sign in to view

github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 25, 2026

MattCarolus10 added 4 commits September 25, 2026 17:32

random.js task fixed

973eda4

variable names changed

e29ee81

number() fixed

a675566

rephrased comments

9cbdc83

MattCarolus10 force-pushed the coursework/sprint-2 branch from 1aca2c1 to 9cbdc83 Compare September 25, 2026 16:38

MattCarolus10 added 5 commits September 25, 2026 18:00

restore Sprint-3 to upstream

a6c7eb1

remove untracked Sprint-3 files

ca8e18f

path exercises fixed

3c63ee6

comment rephrased on 3-to-pounds

a5df0de

correct spelling

31c3d98

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

pad time output to 2 digits

f214a57


		// e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?

		// Answer: ReplaceAll() function clears out all the commas in the strings, the number() converts the strings to numbers to help with calculation


		// f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer

		// Answer: It will work for integer values and will give us clear and clean result using this code. However, the result would not be clean as this using float values

Uh oh!

Conversation

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

Uh oh!

Task code

Changelist

Questions

Uh oh!

netlify Bot commented Sep 19, 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!

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

Uh oh!

Uh oh!

MattCarolus10 commented Sep 24, 2026

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!

Choose a reason for hiding this comment

Uh oh!

This comment has been minimized.

This comment has been minimized.

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

MattCarolus10 commented Sep 19, 2026 •

edited

Loading

netlify Bot commented Sep 19, 2026 •

edited

Loading

hackertainment commented Sep 24, 2026 •

edited

Loading