Skip to content

Update course descriptions#1894

Merged
cifarquhar merged 3 commits into
mainfrom
update-course-descriptions
May 28, 2026
Merged

Update course descriptions#1894
cifarquhar merged 3 commits into
mainfrom
update-course-descriptions

Conversation

@cifarquhar
Copy link
Copy Markdown
Contributor

Finally fixing a couple of things that annoy me every time I look at the home page, and which I have to warn people about in every training session I run:

  • Fixed the ITP course description so it no longer references things we don't teach
  • Removed the duplicate "Post-ITP Tracks" block and renamed the other one "Specialist Tracks"
  • Updated the Checkpoint emoji, because now the fish makes even less sense

@netlify
Copy link
Copy Markdown

netlify Bot commented May 28, 2026

Deploy Preview for cyf-common ready!

Name Link
🔨 Latest commit 2444e68
🔍 Latest deploy log https://app.netlify.com/projects/cyf-common/deploys/6a180c655618db0008728d12
😎 Deploy Preview https://deploy-preview-1894--cyf-common.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@netlify
Copy link
Copy Markdown

netlify Bot commented May 28, 2026

Deploy Preview for cyf-curriculum ready!

Name Link
🔨 Latest commit 2444e68
🔍 Latest deploy log https://app.netlify.com/projects/cyf-curriculum/deploys/6a180c65f8ce2a000879b47a
😎 Deploy Preview https://deploy-preview-1894--cyf-curriculum.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 85 (🔴 down 13 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

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

@cifarquhar cifarquhar marked this pull request as ready for review May 28, 2026 09:35
Copy link
Copy Markdown
Member

@illicitonion illicitonion left a comment

Choose a reason for hiding this comment

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

Generally LGTM, but not sure on removing Post-ITP tracks - the "before checkpoint" vs "after checkpoint" separation does feel useful... WDYT?

@cifarquhar
Copy link
Copy Markdown
Contributor Author

Agree that the separation could be useful, but only if we make a distinction between which tracks are suitable for post-ITP and which are post-SDC. Currently it's the same list in two places, which is more likely to cause confusion.

If we do keep them on a single list I think after checkpoint is better. They're courses designed to support people in finding specific roles, so listing them under "Intro Courses" doesn't feel right to me.

Copy link
Copy Markdown
Member

@illicitonion illicitonion left a comment

Choose a reason for hiding this comment

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

Fair enough! If only we could just express everything in life as a directed graph.

@github-project-automation github-project-automation Bot moved this from 📋 Backlog to 👀 In review in Curriculum Planning May 28, 2026
@cifarquhar cifarquhar merged commit c9c7b3b into main May 28, 2026
11 checks passed
@cifarquhar cifarquhar deleted the update-course-descriptions branch May 28, 2026 12:45
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Curriculum Planning May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants