Skip to content

London | 26-ITP-January | Karla Grajales | Sprint 1 | coursework/sprint-1#910

Open
Grajales-K wants to merge 32 commits intoCodeYourFuture:mainfrom
Grajales-K:coursework/sprint-1
Open

London | 26-ITP-January | Karla Grajales | Sprint 1 | coursework/sprint-1#910
Grajales-K wants to merge 32 commits intoCodeYourFuture:mainfrom
Grajales-K:coursework/sprint-1

Conversation

@Grajales-K
Copy link

@Grajales-K Grajales-K commented Jan 25, 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

This project is about starting to learn how to use methods in JS, and using the console on the browser, at some point, my branch remotely and locally lost the connection, so I ran from my terminal git branch --unset-upstream and then
git push -u origin coursework/sprint-1 to link the branches.

Questions

Do you have any recommendations for learning how to use Git more deeply, especially when we want to go back in history and keep only a specific part in the head of my branch?

Basically, a surgery choosing a specific part to bring back from historical commits.

@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@Grajales-K Grajales-K added 🏕 Priority Mandatory This work is expected 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Submit:PR Module-Structuring-And-Testing-Data The name of the module. labels Jan 25, 2026
@github-actions

This comment has been minimized.

@github-actions github-actions bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jan 26, 2026
@github-actions

This comment has been minimized.

@Theoreoluwa Theoreoluwa added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Feb 14, 2026
@Grajales-K Grajales-K added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 17, 2026
@Grajales-K
Copy link
Author

Very well done so far. Kindly see to the comments made and how to make it better.

Hi @Theoreoluwa, thank you very much for your time reviewing my code. I have made the suggested changes, and I hope it is now more suitable and readable. Thanks!

@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@github-actions github-actions bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 17, 2026
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@Grajales-K Grajales-K added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 17, 2026
@Theoreoluwa Theoreoluwa added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Reviewed Volunteer to add when completing a review with trainee action still to take. labels Feb 18, 2026
Copy link

@Theoreoluwa Theoreoluwa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done, you've done all required so far. You just need to sort out the newly committed file and folder that's not meant for this sprint before I get your PR approved.

@Theoreoluwa Theoreoluwa 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 Feb 18, 2026
@Grajales-K Grajales-K added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Module-Structuring-And-Testing-Data The name of the module. Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 🏕 Priority Mandatory This work is expected Reviewed Volunteer to add when completing a review with trainee action still to take. 📅 Sprint 1 Assigned during Sprint 1 of this module Submit:PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments