-
Notifications
You must be signed in to change notification settings - Fork 59
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
fixing see also links for txt files #796
base: main
Are you sure you want to change the base?
fixing see also links for txt files #796
Conversation
Thanks for the pull request, @jesgreco! This repository is currently maintained by @openedx/docs-openedx-org-maintainers. 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.
|
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.
I don't think these see also's are thorough enough. Best approach would be to find the navigation section that talks about these tasks, and grab the refs from that.
E.g. for add_html, that is most similar to "Add Text Components" from navigation. Grab those links
Changes to the see also tables were made based on navigation files. |
@jswope00 I think after all my changes this PR is ready for Sarina's review and merge. |
fixing see also links for add_video.txt