Skip to content
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

docs: fixing make docs command. #36280

Merged
merged 2 commits into from
Feb 21, 2025
Merged

docs: fixing make docs command. #36280

merged 2 commits into from
Feb 21, 2025

Conversation

awais786
Copy link
Contributor

@awais786 awais786 commented Feb 20, 2025

make docs command still failing on local with duplicate apps erros.

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/edx-platform-docs/envs/latest/lib/python3.11/site-packages/sphinx/config.py", line 529, in eval_config_file
    exec(code, namespace)  # NoQA: S102
    ^^^^^^^^^^^^^^^^^^^^^
  File "/home/docs/checkouts/readthedocs.org/user_builds/edx-platform-docs/checkouts/latest/docs/conf.py", line 34, in <module>
    django.setup()
  File "/home/docs/checkouts/readthedocs.org/user_builds/edx-platform-docs/envs/latest/lib/python3.11/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/docs/checkouts/readthedocs.org/user_builds/edx-platform-docs/envs/latest/lib/python3.11/site-packages/django/apps/registry.py", line 93, in populate
    raise ImproperlyConfigured(
django.core.exceptions.ImproperlyConfigured: Application labels aren't unique, duplicates: content_libraries

@awais786 awais786 marked this pull request as ready for review February 20, 2025 07:32
@awais786 awais786 requested a review from feanil February 20, 2025 09:17
Copy link
Contributor

@feanil feanil left a comment

Choose a reason for hiding this comment

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

This seems fine, but I did add those because I was seeing errors, having trouble re-producing this now though so maybe there was something else up with my local environment. Let's merge this and if it was a heisenbug we'll see it again.

@awais786 awais786 merged commit b5de356 into master Feb 21, 2025
49 checks passed
@awais786 awais786 deleted the fixing-make-docs branch February 21, 2025 08:00
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

gabrielC1409 added a commit to aulasneo/edx-platform that referenced this pull request Feb 21, 2025
…to send to edx-submission

test: Adjust styles

feat: course unit - edit iframe modal window (openedx#35777)

Adds logic to support the functionality of editing xblocks via the legacy modal editing window.

docs: fixing make docs command. (openedx#36280)

* docs: fixing make docs command.

fix: import authoring filter from content_authoring instead (openedx#36109)

Correctly name authoring subdomain according to DDD docs: https://openedx.atlassian.net/wiki/spaces/AC/pages/663224968/edX+DDD+Bounded+Contexts

fix: ADR document update for change from Xqueue to edx-submission
gabrielC1409 added a commit to aulasneo/edx-platform that referenced this pull request Feb 21, 2025
…to send to edx-submission

test: Adjust styles

feat: course unit - edit iframe modal window (openedx#35777)

Adds logic to support the functionality of editing xblocks via the legacy modal editing window.

docs: fixing make docs command. (openedx#36280)

* docs: fixing make docs command.

fix: import authoring filter from content_authoring instead (openedx#36109)

Correctly name authoring subdomain according to DDD docs: https://openedx.atlassian.net/wiki/spaces/AC/pages/663224968/edX+DDD+Bounded+Contexts

fix: ADR document update for change from Xqueue to edx-submission
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.

4 participants