Skip to content

Comments

Barcelona | OCT25 | Tatiana Kuzmina | Sprint 2 | Module-Data-Groups#39

Open
tanibien wants to merge 1 commit intoMigracode-Barcelona:mainfrom
tanibien:coursework/sprint-2
Open

Barcelona | OCT25 | Tatiana Kuzmina | Sprint 2 | Module-Data-Groups#39
tanibien wants to merge 1 commit intoMigracode-Barcelona:mainfrom
tanibien:coursework/sprint-2

Conversation

@tanibien
Copy link

Learners, PR Template

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

Changelist

I have completed the Sprint 2 challenges focusing on JavaScript Objects and Array manipulation.
Changes Made:
Debugged Objects: Fixed property access in address.js and recipe.js, and corrected object iteration in author.js.
Implemented Logic: * Created a tally function for frequency counting.
Built a lookup table from nested arrays.
Developed a contains check for object properties.
Refined Query Parser: Fixed a bug in querystring.js to correctly handle values containing extra = signs using indexOf and slice.
Inverted Objects: Fixed invert.js by using bracket notation for dynamic keys.
Testing:
Ran npm test and all implemented logic passes.

Questions

Ask any questions you have for your reviewer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant