Skip to content

fix: minimize Open edX references#28

Merged
robrap merged 1 commit into
masterfrom
robrap/BOMS-493-update-docs
Apr 8, 2026
Merged

fix: minimize Open edX references#28
robrap merged 1 commit into
masterfrom
robrap/BOMS-493-update-docs

Conversation

@robrap
Copy link
Copy Markdown
Contributor

@robrap robrap commented Apr 8, 2026

  • Update index.html to minimize Open edX references
  • Remove Open edX search
  • Remove old open-source references

Copilot AI review requested due to automatic review settings April 8, 2026 03:04
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR reduces Open edX-related references and removes the legacy site-wide search functionality from the docs landing page and repository metadata/docs.

Changes:

  • Removed the standalone search page and its site list/config documentation.
  • Updated index.html copy and navigation to remove several Open edX-specific audience links and the embedded search widget.
  • Removed Open edX repository metadata and contribution guidance files.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
search.html Deleted the legacy Google Programmable Search–backed search page.
search-sites.txt Deleted the list of sites used to configure the legacy search engine.
README.md Removed search documentation; kept only overview + release notes.
openedx.yaml Removed Open edX repo metadata file.
index.html Updated page title/tagline; removed Open edX audience links and the search sidebar; added a single “Open edX Resources” link.
CONTRIBUTING.md Deleted contribution guidelines referencing Open edX docs processes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread index.html
Comment thread README.md
Comment thread README.md
- Update index.html to minimize Open edX references
- Remove Open edX search
- Remove old open-source references
@robrap robrap force-pushed the robrap/BOMS-493-update-docs branch from 489ffaa to 3ba2b23 Compare April 8, 2026 03:41
Copilot AI review requested due to automatic review settings April 8, 2026 03:41
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 13 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread index.html Outdated
Comment thread course-staff.html
Comment thread researchers.html
Comment thread doroob.html
@robrap robrap force-pushed the robrap/BOMS-493-update-docs branch from 3ba2b23 to 858d4c0 Compare April 8, 2026 03:45
Comment thread index.html
<h2 class="info__title">Feedback or Questions about edX Documentation</h2>
<div class="info__copy">
<p>
edX welcomes and appreciates your comments on and requests for documentation. Just email us at <a href="mailto:docs@edx.org">docs@edx.org</a>.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note to Reviewer: I could also remove this request for feedback.

Comment thread index.html
<footer class="footer--primary" role="contentinfo">
<div class="content--copyright">
<p class="copyright__copy">Copyright &copy; 2018&ndash;2023 <a href="http://www.edx.org">edX Inc.</a>
edX and Open edX are registered trademarks of edX LLC. MicroMasters and MicroBachelors are registered trademarks of Axim Collaborative. All Rights Reserved.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note to Reviewer: I didn't touch this, but I could switch "edX LLC" to "2U". Not sure if the same should happen to "edX Inc." on the line above as well? We could also do nothing, as I did.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fine to leave as is for now.

@robrap
Copy link
Copy Markdown
Contributor Author

robrap commented Apr 8, 2026

Note for Reviewer: This file https://docs.edx.org/doroob.html also exists, as doroob.html. It is also out of date. I'm not sure who these are partners of, and whether this file should simply be removed?

@robrap robrap merged commit d85451b into master Apr 8, 2026
1 check passed
@robrap robrap deleted the robrap/BOMS-493-update-docs branch April 8, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants