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.

@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. Reviewed Volunteer to add when completing a review with trainee action still to take. labels Feb 18, 2026
@Grajales-K
Copy link
Author

Grajales-K commented Feb 24, 2026

Hi @Theoreoluwa, please, if you have a chance, can you kindly review my PR? Thank you

@Theoreoluwa
Copy link

I’ll get that done for you tonight. Had a busy schedule for the past week.

@Grajales-K
Copy link
Author

@Theoreoluwa, no pressure. When you have time, thank you for your time.

@Theoreoluwa Theoreoluwa 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 Feb 24, 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.

Good work, Karla. You utilised the feedback given to improve on your task, which is a very good trait. Well done and keep going!

@Theoreoluwa Theoreoluwa added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed. Module-Structuring-And-Testing-Data The name of the module. 🏕 Priority Mandatory This work is expected 📅 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