Skip to content

London | 26-ITP-Sept | Maharit Belai | Sprint 2 | Course work - #1543

Open
MaharitB wants to merge 36 commits into
CodeYourFuture:mainfrom
MaharitB:coursework/sprint2
Open

London | 26-ITP-Sept | Maharit Belai | Sprint 2 | Course work#1543
MaharitB wants to merge 36 commits into
CodeYourFuture:mainfrom
MaharitB:coursework/sprint2

Conversation

@MaharitB

@MaharitB MaharitB commented Sep 19, 2026

Copy link
Copy Markdown

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

Answered all the questions provided and have fixed bugs of a given code.

Questions

@MaharitB
added label name and email and button
5b7222c
@MaharitB
added <select> for choosing color and size
b07d9e0
@MaharitB
added labe; for color and size
ef3c302
@MaharitB
style body and button for the index.html
7737aa9
@MaharitB
remove comment tags and change footer content
444811f
@MaharitB
change the pattern in input name to .\*S.\*S*.
2cddd89
@MaharitB
remove the comment on line 17.
87ad785
@MaharitB
change pattern of input name in to the .*\S.*\S.*
45a1625
@MaharitB
format document using prettier
cac261e
@MaharitB
add comment to 1-count.js
0692f93
@MaharitB
add ${} in backticks to creat a template literal to get initials from…
d94c63b 
… the strings
@MaharitB
add slice() that would rerturn the dir part of the path
3440700
@MaharitB
declared dotIndex to save the the index of the dot.
7a6f25a
@MaharitB
declared a variable startDirIndex to save the start index of the stri…
f3bef1e 
…ng User
@MaharitB
delet content of slice()and replaced it to StartDirIndex
0fc7d66
@MaharitB
delet 2 notes from the 3-paths.js
9c79a73
@MaharitB
change colon to semicolon
9864337
@MaharitB
remove +1 from slice() to make the output include dot(.)
c8fd52b
@MaharitB
delet all comment lines
4fb372d
@MaharitB
remove the comment lines
c308f99
@MaharitB
explain the expression in order of mathematical operation
c4aa428
@MaharitB
wrote console.log() to see what the code is doing
8d6342a
@MaharitB
format document
e2942bc
@MaharitB
add a single line comment for both lines
1879076
@MaharitB
add a comment on why it produces error
107ef70
@MaharitB
add more to the comment
6e38086
@MaharitB
add comment an answer to a first given question
4ed1c21
@MaharitB
write a console .log() to run the code
1254df6
@MaharitB
edit code by enclosing the number by double quote then it will be tre…
7a98897 
…ated as a string
@MaharitB
edit the comment and delet the comments that were previously there
1289091
@netlify

netlify Bot commented Sep 19, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for cyf-onboarding-module ready!

Name Link
🔨 Latest commit b55c89a
🔍 Latest deploy log https://app.netlify.com/projects/cyf-onboarding-module/deploys/6aae9c7820a53100086dde90
😎 Deploy Preview https://deploy-preview-1543--cyf-onboarding-module.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
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.

@github-actions

This comment has been minimized.

Sign in to view
@MaharitB MaharitB added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 19, 2026
@github-actions

This comment has been minimized.

Sign in to view
@github-actions github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 19, 2026
@MaharitB MaharitB added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MaharitB