London | 26-ITP-Sep | Diana Ausiejute | Sprint 2 | JavaScript Fundamentals by ausiejute · Pull Request #1571 · CodeYourFuture/Module-JavaScript-Fundamentals

ausiejute · 2026-09-22T00:51:37Z

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

Changelist

This branch was recreated from main so that it contains only Sprint-2 coursework.

Fixed everything that was pointed out.

netlify · 2026-09-22T00:51:43Z

✅ Deploy Preview for cyf-onboarding-module ready!

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

abdishakoor-dev

Your answers in 1.js, 2.js and 4.js are clear. Each one has node's message, the reason, and a fix that runs. Prettier passes on every file, too.

Some answers need another look before I can mark this Complete:

1-key-exercises/3-paths.js line 31: check dir against the diagram.
1-key-exercises/4-random.js: break the expression down, step by step.
2-mandatory-errors/3.js: the fix changes the wrong line.
3-mandatory-interpret/1-percentage-change.js: answers a), b) and d).
3-mandatory-interpret/2-time-format.js: answers b), d), e) and f).
3-mandatory-interpret/3-to-pounds.js: lines 29 to 35 say some values are numbers. There is also no step for line 18.
Delete the old code you commented out. Git keeps the old version for you. That is 3-paths.js lines 24 to 30, 1.js lines 3 and 4, 2.js lines 4 and 5, 3.js line 2 and 4.js lines 1 and 2. Also delete the second console.log(result); on line 29 of 2-time-format.js.

See my comment on each line. Add the Needs Review label again once you have pushed.

abdishakoor-dev · 2026-09-22T10:21:51Z

+//const dir = filePath.slice(0 + lastSlashIndex);
+//const dir = filePath.slice(0 + (lastSlashIndex - 1));
+//const dir = filePath.slice(0 + (lastSlashIndex - 44));
+const dir = filePath.slice(0, lastSlashIndex + 1);


Log dir and compare it with the diagram. Your dir ends with a /. Is that last / part of dir? Or is it the separator between dir and base?

The last trailing slash is part of dir, because from what I've found, directory path variables should end with a trailing slash to clearly indicate that they represent directories. Edit. I see, for Unix it's different. Will fix it.

abdishakoor-dev · 2026-09-22T10:21:51Z

 // Try logging the value of num and running the program several times to build an idea of what the program is doing
+
+console.log(num);
+// This expression uses a function that returns a random number between (min)1 and (max)100.


The range is right. The exercise also asks you to break the expression down. What does Math.random() give? What does Math.floor do to it? Which part makes the smallest value 1?

Math.random() gives a decimal between 0 < 1. I multiply it by 100 to stretch it out, then Math.floor() chops off the decimals to make it a whole number. That gives me 0 to 99. The minimum at the end just adds 1 to the whole thing, so now it goes from 1 to 100 instead of 0 to 99

abdishakoor-dev · 2026-09-22T10:21:51Z

+// The actual error indicates that on the 3rd line there is a typeError: "cardNumber.slice is not a function"
+// Checked the error reference and decided to look more closely. Noticed that the card number is used as a number,
+// so it answers why the function couldn't be called - because they can be only called on Arrays and Strings.
+// Therefore, I'll add parentheses to turn the card number into a string (so that the function could be called)


Your explanation of the error is good. But look at line 1. You changed cardNumber itself into a string. The exercise asks you to change the expression on line 3 instead. How can line 3 turn the number into a string?

Interesting. And yes, I need to focus more on what the exercise asks me to do. Fixed it by converting the number into string programmatically grammatically and then used the same method to extract the last 4 numbers

abdishakoor-dev · 2026-09-22T10:21:51Z


 // a) How many function calls are there in this file? Write down all the lines where a function call is made
-
+// 4, 5, 8


Look at line 8 again. Is anything called there? And look at line 10. What is console.log(...)? Also, how many calls are there in total? Line 4 has two.

5 in total?

abdishakoor-dev · 2026-09-22T10:21:51Z

+// 4, 5, 8
 // b) Run the code and identify the line where the error is coming from - why is this error occurring? How can you fix this problem?
-
+// It was a syntax error, it can be fixed by adding the missing part (1 of 2 parentheses)


Node says "missing ) after argument list", but a bracket was not the problem. Compare line 5 with line 4 in the original. What was missing between the two arguments? And which line was it?

abdishakoor-dev · 2026-09-22T10:21:51Z

+// 6
 // b) How many function calls are there?
-
+// 5


Which five? % and / are operators, not function calls. Look for a name followed by (...).

Oh I see. So only the console.log(result) then

abdishakoor-dev · 2026-09-22T10:21:51Z


 // d) Interpret line 4, what does the expression assigned to totalMinutes mean?
-
+// total minutes (in seconds) - remaining seconds = remaining seconds. Then converts the seconds into minutes


Line 4 starts with movieLength, not total minutes. What is 8784 - 24? Why does the program take the 24 seconds away before it divides by 60?

It's movieLength - remainingSeconds. Because those seconds don't form a complete minute, so it's better to remove them in order to avoid a messy decimal

abdishakoor-dev · 2026-09-22T10:21:51Z

+// total minutes (in seconds) - remaining seconds = remaining seconds. Then converts the seconds into minutes
 // e) What do you think the variable result represents? Can you think of a better name for this variable?
-
+// It represents how much of the movie is left to watch. Maybe something like remainderOfTheMovie


Run the file. It prints 2:26:24. movieLength is the length of the whole movie. So is result the time left to watch?

oh, i see, so the result could be replaced into something like totalMovieLength

abdishakoor-dev · 2026-09-22T10:21:51Z

-
+// It represents how much of the movie is left to watch. Maybe something like remainderOfTheMovie
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
+// It won't work with all values. Most importantly, the value must be strictly numeric and positive


Try some values and write down what each one prints. Try 59, -90 and 90.5. Would you show a time that way?

0:0:59 , 0:-1:-30 , 0:1:30.5. Definitely not. I obviously haven't checked the edge cases

abdishakoor-dev · 2026-09-22T10:21:51Z

+// 3-6. const penceStringWithoutTrailingP = penceString.substring(0, penceString.length - 1);
+// : initializes a variable, the value of which is turned to numerical by removing the letter "p".
+// 8. const paddedPenceNumberString = penceStringWithoutTrailingP.padStart(3, "0");
+// Here the padstart function turns "399" string into 399 number (it could add some zeroes in front, but here it serves as a converter)


Does padStart turn "399" into a number? Change line 1 to "5p" and run it. What does each line give now? That shows what padStart and padEnd are for.

Interesting

ausiejute added 9 commits September 21, 2026 21:25

Finished the first 4 key exercises

3aed680

Finished all 4 mandatory errors tasks

6fa81f6

Finished all 3 mandatory interpretation tasks

7456ec0

Finished both stretch-explore tasks

950078c

Completed 4 key exercises

b181b7c

Completed all tasks in mandatory errors folder

9977d32

Completed mandatory interpretation tasks in the folder 3

e18f838

Completed the stretch-explore tasks in folder 4

efc61f0

Remove duplicate Sprint-2 folders

01624f5

ausiejute added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 📅 Sprint 2 Assigned during Sprint 2 of this module labels Sep 22, 2026

ausiejute mentioned this pull request Sep 22, 2026

London| 26-ITP-Sep | Diana Ausiejute | Sprint 2 | Sprint 2 Coursework #1558

Closed

4 tasks

abdishakoor-dev reviewed Sep 22, 2026

View reviewed changes

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

ausiejute added 7 commits September 22, 2026 11:55

Remove trailing slash from dir variable

902061e

Breaking the expression down

5b83030

Change the expression last4Digits is assigned to

e6a26fd

Double check the answers

4f42d5d

Modified the answer to question C

26d443e

Correct the answer about padStart and padEnd

2dd06dd

Correct a few answers

04e573d

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


		// a) How many function calls are there in this file? Write down all the lines where a function call is made

		// 4, 5, 8


		// d) Interpret line 4, what does the expression assigned to totalMinutes mean?

		// total minutes (in seconds) - remaining seconds = remaining seconds. Then converts the seconds into minutes

Uh oh!

Conversation

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

Uh oh!

Learners, PR Template

Task code

Changelist

Uh oh!

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

abdishakoor-dev left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

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

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!

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

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

ausiejute commented Sep 22, 2026 •

edited

Loading

netlify Bot commented Sep 22, 2026 •

edited

Loading

ausiejute Sep 22, 2026 •

edited

Loading

ausiejute Sep 22, 2026 •

edited

Loading