Birmingham|25-ITP-SEP|Hadi Vahidi|Sprint 1|coursework/sprint-1 by HadiVahidi20 · Pull Request #742 · CodeYourFuture/Module-Structuring-and-Testing-Data

HadiVahidi20 · 2025-10-09T08:37:54Z

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

Changelist

I hove done all sprint one of module 2.

Questions

thank you very much

…lity

…ity (exercise 4-random.js)

…tion in 2.js

…slicing

…-to-pounds.js

github-actions · 2025-10-15T09:28:40Z

Your PR couldn't be matched to an assignment in this module.

Please check its title is in the correct format, and that you only have one PR per assignment.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

LonMcGregor

Good start on this sprint's tasks, I have spotted a few areas where you could improve answers further

LonMcGregor · 2025-10-15T09:30:05Z

Sprint-1/3-mandatory-interpret/1-percentage-change.js

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

+/*There are four function calls in total: two in the line


Are you sure there are only 4?

LonMcGregor · 2025-10-15T09:30:51Z

Sprint-1/3-mandatory-interpret/2-time-format.js


 // b) How many function calls are there?

+        /*There are no function calls in this code. Everything here is using operators and expressions, not calling functions.*/


Are you sure there are no functions? I think in both this and the previous task you are missing the same function call.

you are right, I forgot to count the console.log as a function call in both codes. thank you

github-actions · 2025-10-15T09:31:52Z

Your PR couldn't be matched to an assignment in this module.

Please check its title is in the correct format, and that you only have one PR per assignment.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

github-actions · 2025-10-15T09:31:54Z

Your PR couldn't be matched to an assignment in this module.

Please check its title is in the correct format, and that you only have one PR per assignment.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

github-actions · 2025-10-15T09:31:55Z

Your PR couldn't be matched to an assignment in this module.

Please check its title is in the correct format, and that you only have one PR per assignment.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

LonMcGregor · 2025-11-10T09:38:24Z

Great work

HadiVahidi20 added 20 commits September 29, 2025 17:45

Add age calculation functionality and HTML structure for age input

1189fcd

Refactor age calculator layout and improve form structure

78b6a33

addWnetListener Example added

ea977e5

Refactor age calculator layout and improve form structure

22c0fee

percentage exersice added

c346f52

Enhance percentage exercise layout and add area calculation functiona…

ea0fca9

…lity

Add area calculation alert to percentage exercise

5df448f

Add comments to clarify count variable increment logic

ef4f642

second exercise is done using charAt(index)

efa5aa8

Implement dir and ext variable assignments in file path exercise

aa951b4

Refactor random number generation logic and enhance comments for clar…

921ba5a

…ity (exercise 4-random.js)

change the codes to comments in 0.js

e32fceb

mandatory-errors 2 is completed

3b87801

Fix variable initialization error by moving console.log after declara…

24ee904

…tion in 2.js

Fix last4Digits assignment by converting cardNumber to string before …

d51395c

…slicing

percentage exercise is completed

14e15a6

the movie length exercise is done

711385f

Update penceString value and enhance console logging for clarity in 3…

e82c680

…-to-pounds.js

Enhance examples in objects.md to clarify console usage and output

b28011e

Implement to24hourtime function and add initial HTML test page

6c7a0b7

HadiVahidi20 added 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Oct 9, 2025

LonMcGregor added the Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. label Oct 15, 2025

LonMcGregor reviewed Oct 15, 2025

View reviewed changes

HadiVahidi20 added 2 commits November 7, 2025 12:05

yse there 5, one in console.log

0cf3390

correct the answer in the 2-time-format.js

aa53b91

HadiVahidi20 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 Nov 7, 2025

LonMcGregor added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Nov 10, 2025

HadiVahidi20 mentioned this pull request Nov 15, 2025

ITP Sep 2025 | Structuring and testing data | Hadi Vahidi HadiVahidi20/My-Coursework-Planner#7

Open

illicitonion closed this Jan 7, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Birmingham|25-ITP-SEP|Hadi Vahidi|Sprint 1|coursework/sprint-1#742

Birmingham|25-ITP-SEP|Hadi Vahidi|Sprint 1|coursework/sprint-1#742
HadiVahidi20 wants to merge 22 commits intoCodeYourFuture:mainfrom
HadiVahidi20:coursework/sprint-1

HadiVahidi20 commented Oct 9, 2025

Uh oh!

github-actions bot commented Oct 15, 2025

Uh oh!

LonMcGregor left a comment

Uh oh!

LonMcGregor Oct 15, 2025

Uh oh!

LonMcGregor Oct 15, 2025

Uh oh!

HadiVahidi20 Nov 7, 2025

Uh oh!

github-actions bot commented Oct 15, 2025

Uh oh!

github-actions bot commented Oct 15, 2025

Uh oh!

github-actions bot commented Oct 15, 2025

Uh oh!

LonMcGregor commented Nov 10, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants


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

		/*There are four function calls in total: two in the line


		// b) How many function calls are there?

		/There are no function calls in this code. Everything here is using operators and expressions, not calling functions./

Uh oh!

Conversation

HadiVahidi20 commented Oct 9, 2025

Learners, PR Template

Changelist

Questions

Uh oh!

github-actions bot commented Oct 15, 2025

Uh oh!

LonMcGregor left a comment

Choose a reason for hiding this comment

Uh oh!

LonMcGregor Oct 15, 2025

Choose a reason for hiding this comment

Uh oh!

LonMcGregor Oct 15, 2025

Choose a reason for hiding this comment

Uh oh!

HadiVahidi20 Nov 7, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Oct 15, 2025

Uh oh!

github-actions bot commented Oct 15, 2025

Uh oh!

github-actions bot commented Oct 15, 2025

Uh oh!

LonMcGregor commented Nov 10, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants