Skip to content
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

Fix broken link to Code Review explainer #897

Closed
moneyinthesky opened this issue Sep 6, 2024 · 2 comments
Closed

Fix broken link to Code Review explainer #897

moneyinthesky opened this issue Sep 6, 2024 · 2 comments
Assignees
@moneyinthesky
Labels
ITP Work belongs to or is needed for ITP

Comments

@moneyinthesky
Copy link
Contributor

moneyinthesky commented Sep 6, 2024 •
edited
Loading

We have several backlog issues related to Code Review (across the different modules).
It looks like we have some broken links in the issue description which we should fix.
For example, CodeYourFuture/Module-User-Focused-Data#97 points to https://curriculum.codeyourfuture.io/how-our-curriculum-works/sprints/self-evaluate/prep/#code-review which is currently returning a 404.
@moneyinthesky moneyinthesky converted this from a draft issue Sep 6, 2024
@moneyinthesky moneyinthesky added the ITP Work belongs to or is needed for ITP label Sep 6, 2024
@moneyinthesky
Copy link
Contributor Author

We can update the link to https://programming.codeyourfuture.io/how-our-curriculum-works/sprints/self-evaluate/prep/#code-review

@moneyinthesky moneyinthesky self-assigned this Sep 8, 2024
@SallyMcGrath
Copy link
Member

SallyMcGrath commented Sep 20, 2024 •
edited
Loading

This is in Done - is there a related PR? Can you link it @moneyinthesky ?

Edited -- OIC it's a ticket

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

@moneyinthesky moneyinthesky

Labels
ITP Work belongs to or is needed for ITP
Projects
Curriculum Planning
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@moneyinthesky @SallyMcGrath