Skip to content

Cape Town | 2026-ITP-Jan | Isaac Abodunrin | Sprint 2 | Grouping data: Objects#1143

Open
bytesandroses wants to merge 57 commits intoCodeYourFuture:mainfrom
bytesandroses:coursework/Sprint-2
Open

Cape Town | 2026-ITP-Jan | Isaac Abodunrin | Sprint 2 | Grouping data: Objects#1143
bytesandroses wants to merge 57 commits intoCodeYourFuture:mainfrom
bytesandroses:coursework/Sprint-2

Conversation

@bytesandroses
Copy link
Copy Markdown
Member

@bytesandroses bytesandroses commented Mar 28, 2026

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

I completed the following together with tests where applicable:

  • Debug (address.js, author.js, recipe.js)
  • Implement (contains.js, lookup.js, querystring.js, and tally.js)
  • Interpret (invert.js)

@bytesandroses
Implementation for median
3d7370b
@bytesandroses
Test for empty array
fdbd966
@bytesandroses
Test for array without duplicates
a28e904
@bytesandroses
Implement dedupe
06339fc
@bytesandroses
Testing for dedupe
4426ed9
@bytesandroses
Refactor test
4b1ac0d
@bytesandroses
Testing for max
b6c0807
@bytesandroses
Fix: correct case 2 input to array with single number, not just a sin…
aa0f82d 
…gle number (non array)
@bytesandroses
Implement findMax
cc952dd
@bytesandroses
Readability: remove unused code
8596f7f
@bytesandroses
Fix: reduce number of decimal places returned for sum
ebbad7e
@bytesandroses
Refactor to use for of loop
0940103
@bytesandroses
Debug: address.js
1c72bab
@bytesandroses
Debug author
88d95b1
@bytesandroses
Fix: correct for loop
c0f7bb1
@bytesandroses
Fix: correct for loop
cb55db5
@bytesandroses
Fix: print property instead of entire object
81f9946
@bytesandroses
Test: Case 1 -- prop exists in object
1c5c121
@bytesandroses
Fix: correct passing obj and prop into test
b22a517
@bytesandroses
Test: Case 2 for when property not present in object
7689546
@bytesandroses
Test: Case 3 for empty object
ceebce1
@bytesandroses
Test: Case 3 for empty object
b688460
@bytesandroses
Test: Case 4 for invalid objects
fdc221d
@bytesandroses
Implement contains.js
c68b85c
@bytesandroses
Refactor with describe
49e36bb
@bytesandroses
Test: add test for Case 2
7ad5748
@bytesandroses
Refactor: remove unnecessary try/catch
34d7c95
@bytesandroses
Refactor for readability
47113d8
@bytesandroses
Fix: reorder validation checks to prevent weird results
8aefebd
@bytesandroses
Refactor: remove unnecessary try/catch wrapping
ff9eac0
@bytesandroses
Check for nulls and arrays
e6cde3f
@bytesandroses
Refactor
c837ff1
@bytesandroses
Test: empty object input
273dc37
@bytesandroses
Test: array without duplicates
eb1dee9
@bytesandroses
Implementation of tally function
340015c
@bytesandroses
Fix: correct tally variable with counts
420bc1f
@bytesandroses
Test: array with duplicates
8919eac
@bytesandroses
Refactor: remove unnecessary debugging code
8d56810
@bytesandroses
Test: invalid non array inputs
bf09c40
@bytesandroses
Fix: split key-value pair on only first occurence of '='
7e6ad11
@bytesandroses
Refactor: test for additional cases
21df2c2
@bytesandroses
Test: strings with multiple key-value pairs
50e80fd
@bytesandroses
Validate input is string
a9a06c3
@bytesandroses
Ensure that '=' with spaces are processed correctly
774cce1
@bytesandroses
Fix: iterate over tests correctly
9873f85
@bytesandroses
Fix: ensure input is stripped of spaces
cd4b7b5
@bytesandroses
Ensure both key and value are provided
0c144f3
@bytesandroses
Ensure that neither the key nor value is made up entirely of whitespace
a62b0eb
@bytesandroses
Fix: correct implementation and test
35e98cc
@github-actions

This comment has been minimized.

Sign in to view
@bytesandroses bytesandroses added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 28, 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 28, 2026
@bytesandroses bytesandroses added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 28, 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

@bytesandroses