Skip to content

Conversation

@wankoak
Copy link

@wankoak wankoak commented Sep 8, 2025

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

Briefly explain your PR.

Questions

Ask any questions you have for your reviewer.

@wankoak wankoak added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 8, 2025
@github-actions
Copy link

github-actions bot commented Sep 8, 2025

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).

@netlify
Copy link

netlify bot commented Sep 8, 2025

Deploy Preview for cyf-onboarding-module ready!

Name Link
🔨 Latest commit 433cf1e
🔍 Latest deploy log https://app.netlify.com/projects/cyf-onboarding-module/deploys/68beffe756920f0008a80890
😎 Deploy Preview https://deploy-preview-761--cyf-onboarding-module.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
2 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 96 (🔴 down 4 from production)
SEO: 90 (🟢 up 4 from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link

github-actions bot commented Sep 8, 2025

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).

@wankoak wankoak changed the title Birmingham | Sept-ITP | Khor Biel | Sprint-1 | Module-Onboarding Birmingham | Sept-ITP | Khor Biel | Module-Onboarding Sep 8, 2025
@github-actions
Copy link

github-actions bot commented Sep 8, 2025

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).

@wankoak wankoak closed this by deleting the head repository Sep 8, 2025
@Pappyjay157 Pappyjay157 self-assigned this Sep 11, 2025
Copy link

@Pappyjay157 Pappyjay157 left a comment

Choose a reason for hiding this comment

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

Great job on this! I tested it and everything works smoothly. The use of a linked .css file gives the work proper structure. Nice work overall

@Pappyjay157 Pappyjay157 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. labels Sep 14, 2025
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