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

add COURSE_SEARCH_API_URL and CONTENT_FILE_SEARCH_API_URL #2095

Closed
3 tasks done
gumaerc opened this issue Feb 9, 2024 · 0 comments · Fixed by #2109
Closed
3 tasks done

add COURSE_SEARCH_API_URL and CONTENT_FILE_SEARCH_API_URL #2095

gumaerc opened this issue Feb 9, 2024 · 0 comments · Fixed by #2109

Comments

@gumaerc
Copy link
Contributor

gumaerc commented Feb 9, 2024

Description/Context

In order to support the work being done in mitodl/ocw-hugo-themes#1313, two new settings / environment variables need to be added: COURSE_SEARCH_API_URL and CONTENT_FILE_SEARCH_API_URL. These values need to be set in ThemeAssetsPipelineDefinition so that the new search API URL's are baked into the Javascript bundles generated for the site.

Plan/Design

  • Add COURSE_SEARCH_API_URL as a Django setting pulled from the environment with a sane default
  • Add CONTENT_FILE_SEARCH_API_URL as a Django setting pulled from the environment with a sane default
  • In ThemeAssetsPipelineDefinition, inject the values of these settings into the params section of build-ocw-hugo-themes-task
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 a pull request may close this issue.

1 participant