Skip to content

Manchester | 26-ITP-Sep | Gustarv Nchitu | Sprint 3 | Coursework/sprint 3 - #1641

Open
Jestavo1 wants to merge 10 commits into
CodeYourFuture:mainfrom
Jestavo1:coursework/sprint-3
Open

Jestavo1 wants to merge 10 commits into
CodeYourFuture:mainfrom
Jestavo1:coursework/sprint-3

Conversation

@Jestavo1

@Jestavo1 Jestavo1 commented Oct 6, 2026 •

Copy link
Copy Markdown

No description provided.

@netlify

netlify Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cyf-onboarding-module ready!

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

@Jestavo1 Jestavo1 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Oct 6, 2026
@github-actions

This comment has been minimized.

@github-actions github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Oct 6, 2026
@Jestavo1 Jestavo1 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Oct 6, 2026
@Poonam-raj Poonam-raj 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 Oct 7, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great explanations - nice refactor of the code

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again good accurate explanations and rework here

Comment on lines +27 to +30
// Explanation: The original getLastDigit function did not have
// a parameter, so the numbers 42, 105, and 806 passed to the
// function were ignored. Instead, the function always used the
// global value 103, so it returned 3 every time.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice explanation here well done

@Poonam-raj Poonam-raj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work here, nice understanding shown in your explanations throughout

@Poonam-raj Poonam-raj added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Oct 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants