Skip to content

feat: [FC-86] Rework config endpoint to use for LMS frontend - #2662

Merged
NiedielnitsevIvan merged 5 commits into
rg/axm-course-catalog-index-config-pagefrom
rg/axm-lms-pages-config-api
Jul 3, 2025
Merged

NiedielnitsevIvan merged 5 commits into
rg/axm-course-catalog-index-config-pagefrom
rg/axm-lms-pages-config-api

Conversation

@Serj-N

@Serj-N Serj-N commented Jul 1, 2025

Copy link
Copy Markdown

This PR expands the frontend configuration API to make it usable for different LMS pages.

  • Include additional keys/values in the response
  • Rename url path & name
  • Rename view
  • Adjust docstrings
  • Adjust tests

@Serj-N
Serj-N requested a review from NiedielnitsevIvan July 1, 2025 09:48
@Serj-N Serj-N self-assigned this Jul 1, 2025
@Serj-N Serj-N changed the title Rework config endpoint to use for LMS frontend feat: [FC-86] Rework config endpoint to use for LMS frontend Jul 1, 2025
@Serj-N
Serj-N requested a review from SergiiKalinchuk July 2, 2025 13:33
Comment thread lms/djangoapps/branding/views.py Outdated
Comment thread lms/djangoapps/branding/views.py Outdated
@NiedielnitsevIvan
NiedielnitsevIvan merged commit 613cd82 into rg/axm-course-catalog-index-config-page Jul 3, 2025
@NiedielnitsevIvan
NiedielnitsevIvan deleted the rg/axm-lms-pages-config-api branch July 3, 2025 09:33
Serj-N added a commit that referenced this pull request Jul 3, 2025
* feat: rework config endpoint to use for different lms pages
# Conflicts:
#	lms/djangoapps/branding/api_urls.py
#	lms/djangoapps/branding/tests/test_views.py
#	lms/djangoapps/branding/views.py
Serj-N added a commit that referenced this pull request Aug 1, 2025
* feat: rework config endpoint to use for different lms pages
Serj-N added a commit that referenced this pull request Sep 9, 2025
* feat: rework config endpoint to use for different lms pages
Serj-N added a commit that referenced this pull request Sep 9, 2025
* feat: rework config endpoint to use for different lms pages
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