fix: added forum requirement for edge#337
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the edxapp Ansible role defaults to ensure Edge/sandbox deployments install the forum package as part of the private Python requirements.
Changes:
- Add
openedx-forumas a private requirement sourced from theedx/forumrepository. - Normalize the
edx-arch-experimentsrequirement line (but introduces trailing whitespace).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
naincy128
approved these changes
Apr 27, 2026
There was a problem hiding this comment.
Pull request overview
Updates the edxapp Ansible role defaults so Edge/sandbox deployments install the forum package via EDXAPP_PRIVATE_REQUIREMENTS.
Changes:
- Add
openedx-forumas an editable git requirement sourced fromedx/forum. - (Per PR description) intends to normalize
edx-arch-experimentsrequirement formatting, though that change is not present in the provided diff.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+575
to
+576
| # forum - currently pointing to ulmo but needs an upgrade when in verawood | ||
| - name: git+https://github.com/edx/forum.git@release-ulmo#egg=openedx-forum |
vgulati-apphelix
force-pushed
the
COSMO2-897
branch
from
a6e845e to
3f58992
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Updated the edxapp Ansible role defaults to ensure Edge/sandbox deployments install the forum package as part of the private Python requirements.
Changes:
Add openedx-forum as a private requirement sourced from the edx/forum repository.
Normalize the edx-arch-experiments requirement line