Skip to content

London | 26-ITP-May | Chun Yan Wong | Sprint 2 | Coursework - #1367

Open
cywong-dev wants to merge 16 commits into
CodeYourFuture:mainfrom
cywong-dev:Sprint-2
Open

London | 26-ITP-May | Chun Yan Wong | Sprint 2 | Coursework#1367
cywong-dev wants to merge 16 commits into
CodeYourFuture:mainfrom
cywong-dev:Sprint-2

Conversation

@cywong-dev

Copy link
Copy Markdown

Learners, PR Template

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

Changelist

This PR provides the my answers for the Sprint 2 - Coursework

Replaced for-of loop with for-in loop to iterate over object properties.
Uncommented the loop to display ingredients and removed old console log.
Add validation to the contains function to check if the input is a valid object and if the property exists.
Implement createLookup function to return country-currency pairs.
Enhanced the parseQueryString function to handle empty query strings and duplicate keys. Improved decoding of parameters and ensured proper handling of key-value pairs.
Added tests for handling duplicate keys in query strings.
Added input validation and tallying functionality.
Fix invert function to correctly invert object key-value pairs.
Add input validation and handle empty strings in countWords function.
Fixed the totalTill function implementation to correctly calculate the total amount in pounds from the till object. Added a test case to verify the functionality.
@cywong-dev cywong-dev added 📅 Sprint 2 Assigned during Sprint 2 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant