-
-
Notifications
You must be signed in to change notification settings - Fork 118
Codewars success criteria amendments #1699
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.
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for cyf-tracks ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-common ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-itd ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-sdc ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-programming ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-piscine ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-curriculum ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for cyf-launch ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
illicitonion
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we not adding to Onboarding and S&TD as well?
illicitonion
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also FYI in the issue template, if you fill the boxes without spaces ([x]) they will get ticked cutely :)
Co-authored-by: Daniel Wagner-Hall <daniel@codeyourfuture.io>
Co-authored-by: Daniel Wagner-Hall <daniel@codeyourfuture.io>
…nto codewars-kata-rejig-praj
…rFuture/curriculum into codewars-kata-rejig-praj
|
S&TD looks good now - are we adding a requirement to do the Onboarding collection during onboarding? Regardless, wherever the first time we ask them for evidence, we should probably include a link to https://codeyourfuture.github.io/Codewars-Progress-Checker/ telling them to put their username in and copy the link? Otherwise I'm not sure people will know how to share that evidence? |
|
Completely discounted onboarding this is a fair point. To be clear, is it that codewars kata evidence is a link to the Codewars Progress Checker with their username put in or is it a screenshot of that? |
They should include the link in their issue, and the person checking it can follow the link to check. (Hopefully this will get automated by the new dashboard soon!) |

What does this change?
Success criteria for Data Groups and Data Flows (ITP) to reflect the fact that katas are now split by sprint.
Issue number: #1620
Checklist
Who needs to know about this?
@illicitonion