Skip to content

feat: enable marketing site feature in CMS and LMS configurations #176

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.

Sign up for GitHub

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

Merged
ktyagiapphelix2u merged 8 commits into from
Oct 27, 2025
Merged

feat: enable marketing site feature in CMS and LMS configurations #176

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
b20277b
feat: enable marketing site feature in CMS and LMS configurations
ktyagiapphelix2u Sep 16, 2025
b6bb4cd
feat: enable marketing site feature in CMS and LMS configurations
ktyagiapphelix2u Sep 18, 2025
10c6ba7
feat: enable marketing site feature in CMS and LMS configurations
ktyagiapphelix2u Sep 19, 2025
51643be
feat: enable marketing site feature in CMS and LMS configurations
ktyagiapphelix2u Sep 22, 2025
70256f3
feat: enable marketing site feature in CMS and LMS configurations
ktyagiapphelix2u Sep 22, 2025
63032f6
feat: enable marketing site feature in CMS and LMS configurations
ktyagiapphelix2u Sep 23, 2025
b96cdad
feat: enable marketing site feature in CMS and LMS configurations
ktyagiapphelix2u Sep 24, 2025
fba9cd3
feat: enable marketing site feature in CMS and LMS configurations
ktyagiapphelix2u Sep 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 11 additions & 2 deletions configuration_files/cms.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ FEATURES:
ENABLE_EXPORT_GIT: false
ENABLE_GRADE_DOWNLOADS: true
ENABLE_LTI_PROVIDER: false
ENABLE_MKTG_SITE: false
ENABLE_MKTG_SITE: true
Copy link
Copy Markdown
Contributor

@robrap robrap Sep 24, 2025

Choose a reason for hiding this comment

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

I thought the FEATURES dict was going away and all of these were duplicated to top-level?

Copy link
Copy Markdown
Contributor Author

@ktyagiapphelix2u ktyagiapphelix2u Sep 26, 2025

Choose a reason for hiding this comment

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

i have fixed them and make them to their correct place

ENABLE_MOBILE_REST_API: false
ENABLE_OAUTH2_PROVIDER: false
ENABLE_PUBLISHER: false
Expand Down Expand Up @@ -382,7 +382,16 @@ LOG_DIR: /edx/var/log/edx
MAINTENANCE_BANNER_TEXT: Sample banner message
MEDIA_ROOT: /edx/var/edxapp/media/
MEDIA_URL: /media/
MKTG_URLS: {}
MKTG_URLS:
ROOT: https://localhost:18010
Comment thread
robrap marked this conversation as resolved.
Outdated
ABOUT: /about
CONTACT: /contact
FAQ: /help
COURSES: /course
TOS: /edx-terms-service
HONOR: /honor
PRIVACY: /privacy
WHAT_IS_VERIFIED_CERT: /verified-certificate
MKTG_URL_LINK_MAP: {}
MOBILE_STORE_ACE_URLS: {}
MODULESTORE:
Expand Down
13 changes: 11 additions & 2 deletions configuration_files/lms.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -346,7 +346,7 @@ FEATURES:
ENABLE_EXPORT_GIT: false
ENABLE_GRADE_DOWNLOADS: true
ENABLE_LTI_PROVIDER: false
ENABLE_MKTG_SITE: false
ENABLE_MKTG_SITE: true
ENABLE_MOBILE_REST_API: false
ENABLE_OAUTH2_PROVIDER: false
ENABLE_PUBLISHER: false
Expand Down Expand Up @@ -422,7 +422,16 @@ MAILCHIMP_NEW_USER_LIST_ID: null
MAINTENANCE_BANNER_TEXT: Sample banner message
MEDIA_ROOT: /edx/var/edxapp/media/
MEDIA_URL: /media/
MKTG_URLS: {}
MKTG_URLS:
ROOT: /https://localhost:18000
ABOUT: /about
CONTACT: /contact
FAQ: /help
COURSES: /course
TOS: /edx-terms-service
HONOR: /honor
PRIVACY: /privacy
WHAT_IS_VERIFIED_CERT: /verified-certificate
MKTG_URL_LINK_MAP: {}
MOBILE_STORE_URLS: {}
MODULESTORE:
Expand Down
2 changes: 1 addition & 1 deletion py_configuration_files/lms.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -424,7 +424,7 @@ def should_show_debug_toolbar(request): # lint-amnesty, pylint: disable=missing
'ENABLE_ENTERPRISE_INTEGRATION': True,
})

ENABLE_MKTG_SITE = os.environ.get('ENABLE_MARKETING_SITE', False)
ENABLE_MKTG_SITE = os.environ.get('ENABLE_MARKETING_SITE', True)
MARKETING_SITE_ROOT = os.environ.get('MARKETING_SITE_ROOT', 'http://localhost:8080')

MKTG_URLS = {
Expand Down