Skip to content

London|26-ITP-January|Alexandru Pocovnicu|Sprint 2|Data Groups#967

Open
alexandru-pocovnicu wants to merge 31 commits intoCodeYourFuture:mainfrom
alexandru-pocovnicu:Sprint-2
Open

London|26-ITP-January|Alexandru Pocovnicu|Sprint 2|Data Groups#967
alexandru-pocovnicu wants to merge 31 commits intoCodeYourFuture:mainfrom
alexandru-pocovnicu:Sprint-2

Conversation

@alexandru-pocovnicu
Copy link

@alexandru-pocovnicu alexandru-pocovnicu commented Feb 27, 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

In this PR I have predicted, explained, and fixed bugs, implemented functions according to requrements, and interpreted program with unfamiliar syntax

@alexandru-pocovnicu
correct access to houseNumber in address object
5afdf82
@alexandru-pocovnicu
correct iteration method to log author properties
8a0b3af
@alexandru-pocovnicu
update recipe logging to display ingredients on new lines
f820399
@alexandru-pocovnicu
implement contains function and add tests for empty and existing prop…
6e37fe5 
…erties
@alexandru-pocovnicu
fix contains function to correctly check for property existence in ob…
4fd1b2d 
…ject
@alexandru-pocovnicu
fix contains function to correctly return false for empty objects
400346f
@alexandru-pocovnicu
add test case for contains function with non-existing property name
6e91927
@alexandru-pocovnicu
add error handling test for contains function with invalid object type
06b3767
@alexandru-pocovnicu
remove commented-out console.log statement from contains function
bbb05bc
@alexandru-pocovnicu
implement test for createLookup function to validate country-currency…
8f746c1 
… code mapping
@alexandru-pocovnicu
implement createLookup function to map country codes to currency codes
202e093
@alexandru-pocovnicu
refactor parseQueryString function to improve key-value extraction logic
1a3329e
@alexandru-pocovnicu
fix: handle edge cases in parseQueryString function and update tests
3796cea
@alexandru-pocovnicu
test: add case for empty key in querystring parsing
0c37eb3
@alexandru-pocovnicu
fix: clean up querystring parsing logic and improve test formatting
613c044
@alexandru-pocovnicu
test: add tests for empty array and invalid input in tally function
c279e01
@alexandru-pocovnicu
feat: implement tally function to count occurrences in an array
b93588f
@alexandru-pocovnicu
test: fix assertion in tally function tests and ensure proper formatting
8acd98b
@alexandru-pocovnicu
fix: add input validation to tally function to ensure array input
327ba71
@alexandru-pocovnicu
fix: improve formatting and consistency in tally function and tests
48a17dc
@alexandru-pocovnicu
fix: correct implementation of invert function to properly swap keys …
46c15fa 
…and values, add comments to explain the process
@alexandru-pocovnicu
test: add initial test suite for invert function
b669259
@alexandru-pocovnicu
fix: add input validation to invert function to handle non-object inp…
4c94700 
…uts and improve error handling
@alexandru-pocovnicu
fix: update invert function test to handle null input and ensure prop…
b80bd10 
…er error handling
@alexandru-pocovnicu
test: add additional test cases for invert function to validate behav…
5b1dab3 
…ior with various inputs
@alexandru-pocovnicu
fix: improve input validation in invert function and update test case…
9935333 
…s for consistency
@alexandru-pocovnicu
refactor: split calculateMode into smaller functions for improved rea…
12de278 
…dability
@alexandru-pocovnicu alexandru-pocovnicu added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 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

@alexandru-pocovnicu