Skip to content

London | 26-ITP-May | Alex Jamshidi | Sprint 2 | Book Library#496

Open
Alex-Jamshidi wants to merge 1 commit into
CodeYourFuture:mainfrom
Alex-Jamshidi:book-library
Open

London | 26-ITP-May | Alex Jamshidi | Sprint 2 | Book Library#496
Alex-Jamshidi wants to merge 1 commit into
CodeYourFuture:mainfrom
Alex-Jamshidi:book-library

Conversation

@Alex-Jamshidi
Copy link
Copy Markdown

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

Fixed bugs in book library app

@Alex-Jamshidi Alex-Jamshidi added 📅 Sprint 2 Assigned during Sprint 2 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Module-Data-Flows The name of the module. labels May 14, 2026
@RaihanSharif
Copy link
Copy Markdown

RaihanSharif commented May 14, 2026

Check your work against this guide:

https://github.com/CodeYourFuture/Module-Data-Flows/blob/general-review-feedback/debugging/book-library/feedback.md

I've also removed the needs review label for now, you can add it back once you've made your changes and want it reviewed.

@RaihanSharif RaihanSharif removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label May 14, 2026
@Alex-Jamshidi
Copy link
Copy Markdown
Author

Thank you for this, really helpful

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

Labels

Module-Data-Flows The name of the module. 📅 Sprint 2 Assigned during Sprint 2 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants