Skip to content

London | 26-ITP-May | Jorvan White | Sprint 1 | Data Groups - #1452

Open
JorvanW wants to merge 12 commits into
CodeYourFuture:mainfrom
JorvanW:coursework/sprint-1
Open

London | 26-ITP-May | Jorvan White | Sprint 1 | Data Groups#1452
JorvanW wants to merge 12 commits into
CodeYourFuture:mainfrom
JorvanW:coursework/sprint-1

Conversation

@JorvanW

@JorvanW JorvanW commented Aug 21, 2026

Copy link
Copy Markdown

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

Created Pull Request from Sprint 1 Task and updated for Pull Request

@github-actions

This comment has been minimized.

@JorvanW JorvanW added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Aug 21, 2026
@hackertainment hackertainment added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Aug 21, 2026
@hackertainment

hackertainment commented Aug 21, 2026

Copy link
Copy Markdown

There are 4 fixes required. Please also pay more attention to indentation, matching open and closing brackets. The VS Code prettier plugin can help on that. Thank you.

@hackertainment hackertainment added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Reviewed Volunteer to add when completing a review with trainee action still to take.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants