docs: design workflow for xblock skill tagging - #140
Conversation
|
Thanks for the pull request, @navinkarkera! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
21aefad to
8fcdc51
Compare
|
@sameenfatima78 Please let me know your thoughts on the document as well as whether this is a good place to keep it. |
8fcdc51 to
8b7e77c
Compare
robrap
left a comment
There was a problem hiding this comment.
- This is immensely helpful. Thank you!
- Were there any Rejected Alternatives to document?
- I’ll leave the more detailed review to the teams involved.
|
Hi @moconnell1453! Flagging this for your review. |
8b7e77c to
8149528
Compare
|
@robrap Thank you!
No. |
|
@moconnell1453 Thank you! I cannot merge it as I don't have merge rights. |
sameenfatima78
left a comment
There was a problem hiding this comment.
Looks awesome @navinkarkera, thanks for this!
@navinkarkera I'll take care of it. |
|
@navinkarkera 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Adds document describing the whole workflow for xblock skill tagging.
Merge checklist:
requirements/*.txtfiles)make upgrade && make requirementshave been run to regenerate requirements./manage.py makemigrationshas been runPost merge:
(so basically once your build finishes, after maybe a minute you should see the new version in PyPi automatically (on refresh))
make upgradein course-discovery will look for the latest version in PyPi.