-
Notifications
You must be signed in to change notification settings - Fork 5
docs: added github access details to CC onboarding course #72
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
|
Thanks for the pull request, @MahnoorArbisoft! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
|
@MahnoorArbisoft when this is unpacked it's really messed up, I'm not sure what happened to your exported course here - you only changed one file, right? I can't actually see what you changed. Could you point to a course with the updated unit(s), or send screenshots? |
|
Also your title of your PR says you changed the CC onboarding course, but the export is from the Intro to Open edX course - maybe that's the issue? |
|
@cassiezamparini could you help me clarify where this updated text is supposed to be added? |
|
@sarina Is there an easy way to see the course content somewhere on GitHub? Perhaps you can tell me what to do if it's easy enough. My memory of the course content is murky so I need a refresher for context. Or perhaps I should just look at the actual onboarding course, but I don't seem to have access to anymore (even tried resetting my password). Could you give me access? @MahnoorArbisoft Do you have any ideas where this might best land (as you've done the course recently)? |
|
You can download all the courses here: https://github.com/openedx/training-courses/tree/main/dist and import it into your instance. I'm a bit confused as to what course this change is meant to go into. |
|
@sarina This is for the Core Contributor Onboarding Course: https://training.openedx.io/courses/course-v1:OpenedX+NewCC+2024/about I'll wait on @MahnoorArbisoft to comment before trying to import the courses into an instance (I'd need to learn how to do this -- it's a good thing to know). Having access to the actual learner view would be far easier and quicker for me. |
|
Do you have an instance you can import the course into to test it out? Otherwise I need the email address you have an account on the training sit ewith |
|
@sarina I'm not sure why the intro would have been updated. @MahnoorArbisoft can you advise how you went about this / where you intended this to be added in the course? |
|
sorry about the confusion. Not sure what went wrong here. I cant trail back to the issue either because i used our sandbox and that flushes the data everyday. I added this text in "Finding meaningful work" unit of the course but since this was my first PR maybe i did something wrong in the process. I can make changes and export the course again -- would that make it easier? |
|
@MahnoorArbisoft could you just clarify:
|

Based on conversation in this thread: Added details of accessing Github in CC Onboarding Courses
Details of updated Text