[lamlakamana-oss]

Self checklist

• [x ] I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• [x ] My changes meet the requirements of the task
• [x ] I have tested my changes
• [x ] My changes follow the style guide

Task Code

CYF-1053

Completed all mandatory and stretch tasks for Sprint 3.

Fixed bugs in the debugging exercises.

Implemented the toPounds function.

Added edge case tests for the time format exercise.

I agree to follow the code of conduct for this organisation

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

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

[Dilangi]

Hi Lamla, thanks for submitting this. Before the code feedback, a note on the PR itself.

PR title
It's a best practice to use a consistent title format so PRs are easy to track, search and review later. Suggested format:
Region | Cohort | FirstName LastName | Sprint | Assignment Title
Cohort: 25-ITP-Sep

PR description
Please add a "Change list" section before listing down what you changed.

Example:

Change list

You can edit the title and description at any time from the PR page.

[Stephanieewelu]

Thanks for working through these exercises. I think there are still two important issues to fix before this is ready to merge:

• In Sprint-3/1-key-errors/0.js, the function parameter str is redeclared with let str inside the same scope:

• function capitalise(str) { let str = ... }

• This is a SyntaxError and the code is still broken.

• In Sprint-3/1-key-errors/1.js, the same pattern happens with decimalNumber:

• function convertToPercentage(decimalNumber) { const decimalNumber = ... }

• This also causes a redeclaration error.

These files are meant to demonstrate the bug and the fix, so the corrected version should no longer contain the original error. Please rename the inner variables (for example, result / percentage) so the functions work correctly.

I’d suggest fixing those two files and then rechecking the PR.

[lamlakamana-oss]

Renamed the inner variables in 0.js and 1.js. Both files now run without errors.

[cjyuan]

Changes look good. I will mark this PR as complete first.

---

Your PR description is not yet formatted correctly.
Could you reformat it to match the formatting used in this PR?#1618

In particular your PR description is missing a "Changelist" heading, and the checkboxes are incorrectly formatted.

Incorrect syntax

- [x ] I have tested my changes

produces

• [x ] I have tested my changes

Correct syntax

- [x] I have tested my changes

produces

• I have tested my changes