Skip to content

Adding deprecated course redirects#2884

Merged
adamwoodnz merged 2 commits intotrunkfrom
2868-add-course-deprecation-redirects
Sep 16, 2024
Merged

Adding deprecated course redirects#2884
adamwoodnz merged 2 commits intotrunkfrom
2868-add-course-deprecation-redirects

Conversation

@jonathanbossenger
Copy link
Copy Markdown
Collaborator

Fixes #2868

@jonathanbossenger
Copy link
Copy Markdown
Collaborator Author

@adamwoodnz I'm not sure what you think of what I did here, but I thought it would make sense to group redirects by type.

@adamwoodnz
Copy link
Copy Markdown
Contributor

@adamwoodnz I'm not sure what you think of what I did here, but I thought it would make sense to group redirects by type.

I'm not opposed to doing this if you think there's value in organising them. Left some comments about going further with the approach if so.

@jonathanbossenger
Copy link
Copy Markdown
Collaborator Author

Thanks @adamwoodnz PR updated based on your feedback.

Copy link
Copy Markdown
Contributor

@adamwoodnz adamwoodnz left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

@adamwoodnz adamwoodnz merged commit ed7c0cc into trunk Sep 16, 2024
@adamwoodnz adamwoodnz deleted the 2868-add-course-deprecation-redirects branch September 16, 2024 23:59
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.

Course deprecation

2 participants