Skip to content

London | 26-ITP-January | Laura C | Sprint 2 | Data Groups#979

Open
lauracs24 wants to merge 12 commits intoCodeYourFuture:mainfrom
lauracs24:sprint-2
Open

London | 26-ITP-January | Laura C | Sprint 2 | Data Groups#979
lauracs24 wants to merge 12 commits intoCodeYourFuture:mainfrom
lauracs24:sprint-2

Conversation

@lauracs24
Copy link

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

Debug

  • Fixed address.js by accessing object property
  • Fixed author.js by iterating through object keys
  • Fixed recipe.js to correctly display the nice ingredients

Implement

  • Implemented contains function
  • Implemented lookup function
  • Implemented tally function
  • Fixed querystring implementation to handle values containing "="

Interpret

  • Implemented invert to swap keys and values
  • Answered interpretation questions in comments in the exercise

All tests pass successfully using npm test.

@lauracs24 lauracs24 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 4, 2026
@lauracs24 lauracs24 moved this to Awaiting Approval🚀 in Coursework Planner Mar 4, 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant