Skip to content

fix: adds content.search to docs apps (Sumac backport) - #35778

Merged
cmltaWt0 merged 2 commits into
open-release/sumac.masterfrom
sarina/update-docs-config
Nov 6, 2024
Merged

fix: adds content.search to docs apps (Sumac backport)#35778
cmltaWt0 merged 2 commits into
open-release/sumac.masterfrom
sarina/update-docs-config

Conversation

@sarina

@sarina sarina commented Nov 5, 2024

Copy link
Copy Markdown
Contributor
  • fix: adds content.search to docs apps

Description

Adds openedx.core.djangoapps.content.search to the list of installed applications used by sphinx to generate the platform documentation. This app is included in the CMS django config, not the LMS, and so triggers an error without this change.

Supporting information

This is a backport from master to fix the docs build.

See #35766

cc @pomegranited

@sarina
sarina changed the base branch from master to open-release/sumac.master November 5, 2024 17:10
@sarina sarina changed the title fix: adds content.search to docs apps fix: adds content.search to docs apps (Sumac backport) Nov 5, 2024

@pomegranited pomegranited left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

  • I tested this on my sumac devstack
  • I read the commit to check that it matches the original merged #35766

@cmltaWt0
cmltaWt0 merged commit 7ecd6a5 into open-release/sumac.master Nov 6, 2024
@cmltaWt0
cmltaWt0 deleted the sarina/update-docs-config branch November 6, 2024 09:09
@pomegranited

Copy link
Copy Markdown
Contributor

Thank you for getting this done and into sumac @sarina and @cmltaWt0 !

FYI @feoh

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