Skip to content

LONDON-JAN-2026 | Said Fayaz Sadat | Sprint-3 | Data flows#438

Open
fayaz551 wants to merge 3 commits intoCodeYourFuture:mainfrom
fayaz551:book-library
Open

LONDON-JAN-2026 | Said Fayaz Sadat | Sprint-3 | Data flows#438
fayaz551 wants to merge 3 commits intoCodeYourFuture:mainfrom
fayaz551:book-library

Conversation

@fayaz551
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 and implemented the fixes

Questions

n/a

- Corrected variable reference from 'library' to 'myLibrary'.
- Fixed logic bug where title input was being used for the author field.
- Fixed delete button functionality and event listener naming.
- Corrected read status display logic to show "Yes" when checked.
- Changed input types for title and author to 'text' for standard validation.
- Ensured script and bootstrap links are correctly ordered for DOM loading.
- Adjusted form-group dimensions for better visibility.
- Added spacing and alignment for table action buttons.
@github-actions
Copy link
Copy Markdown

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

@fayaz551 fayaz551 changed the title LONDON | JAN - 2026 | Said Fayaz Sadat | Book library LONDON-JAN-2026 | Said Fayaz Sadat | Sprint-3 | Data flows Apr 15, 2026
@github-actions
Copy link
Copy Markdown

Your PR couldn't be matched to an assignment in this module.

Please check its title is in the correct format, and that you only have one PR per assignment.

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant