Skip to content

[RfR] New course guidance #252

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

[RfR] New course guidance #252

wants to merge 5 commits into from

Conversation

liamjberrisford
Copy link
Collaborator

@liamjberrisford liamjberrisford commented May 9, 2025 •
edited
Loading

Closes #234

@liamjberrisford liamjberrisford mentioned this pull request Jul 3, 2025
Open
@liamjberrisford liamjberrisford self-assigned this Jul 16, 2025
felicityguest
felicityguest requested changes Aug 8, 2025
View reviewed changes
Copy link
Contributor

@felicityguest felicityguest left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thank you for putting this guidance together, @liamjberrisford - it's going to prove very useful. I think we could simplify it to more easily ensure adherence to the timeline, so I've included some comments to that effect. If you'd like to discuss any aspects, please do let me know.

@liamjberrisford
Copy link
Collaborator Author

@felicityguest I’ve addressed the outstanding queries and consolidated the checklist and timeline into a single, streamlined document to more clearly reflect each step. I’ve also removed items rendered obsolete by the recent website updates, for example, manual accessibility checks are now automated through our CI pipeline. Would you mind taking another look? Apologies that some of it has changed a fair amount!

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 8, 2025 •
edited
Loading

🚦 Pa11y Accessibility Report

The config file "/__w/CfRR_Courses/CfRR_Courses/pa11yci" could not be loaded

⚠️ Any errors likely originate from the source file (e.g., filename.md, filename.ipynb).

@liamjberrisford
Copy link
Collaborator Author

@felicityguest The changes discussed have now been implemented and the content is ready to be reviewed again.

@liamjberrisford liamjberrisford changed the title New course guidance [RfR] New course guidance Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@felicityguest felicityguest Awaiting requested review from felicityguest

Requested changes must be addressed to merge this pull request.

Assignees

@liamjberrisford liamjberrisford

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Pre-Flight Checklist for New Course Integration
2 participants
@liamjberrisford @felicityguest