Skip to content

Cape Town | 26-ITP-Jan | Pretty Taruvinga | Sprint 2 | Data Groups#1109

Open
Pretty548 wants to merge 18 commits intoCodeYourFuture:mainfrom
Pretty548:sprint-2
Open

Cape Town | 26-ITP-Jan | Pretty Taruvinga | Sprint 2 | Data Groups#1109
Pretty548 wants to merge 18 commits intoCodeYourFuture:mainfrom
Pretty548:sprint-2

Conversation

@Pretty548
Copy link
Copy Markdown

@Pretty548 Pretty548 commented Mar 26, 2026 •
edited
Loading

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

  • Fixed parseQueryString implementation and handled edge cases
  • Fixed contains function bug
  • Implemented tally function with validation
  • Fixed calculateMedian to handle invalid inputs

Pretty Taruvinga added 14 commits March 7, 2026 14:25
implement correct median calculation
53fa90f
Access houseNumber correctly from address object
7b40348
Use Object.values to loop through author object
c04e2dc
display ingredients correctly instead of object
59fff2a
Add contains function to check if value exists in array
bbdc491
implement contains function for object property check
904055c
implement contains test cases
7f28769
Made it look clean
e6ec9d4
Implement createLookup function for country-currency pairs
067f3e1
set up jest and add test script
5b88737
1. add createLookup test cases for multiple and empty inputs
92c0f04 
2. correct import path for createLookup in tests
3.add jest and configure test script.
Correctly parse query strings with '=' in values and edge cases"
12f79f0
Implement tally function with error handling and tests"
3c7d554
Correctly invert object keys and values using bracket notation"
78d5e44
@github-actions

This comment has been minimized.

Sign in to view
4 similar comments
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@Pretty548 Pretty548 changed the title Sprint 2 Cape Town | 26-ITP-Jan | Pretty Taruvinga | Sprint 2 | Data Groups Mar 26, 2026
@github-actions

This comment has been minimized.

Sign in to view
@Pretty548 Pretty548 marked this pull request as draft March 27, 2026 11:32
@Pretty548 Pretty548 marked this pull request as ready for review March 27, 2026 11:32
@Pretty548 Pretty548 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 27, 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 Mar 27, 2026
@Pretty548 Pretty548 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 27, 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 Mar 27, 2026
@Pretty548 Pretty548 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 27, 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

@Pretty548