Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Add PATH_PREFIX configuration #1136

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Add PATH_PREFIX configuration #1136

wants to merge 2 commits into from

Commits on Feb 9, 2023

  1. Add a PATH_PREFIX variable 

    This allows us to "easily" deploy Django to a path prefix but has a ton
of caveats and pitfalls that need to be addressed in subsequent commits.
Specifically: tests will not work with the current definition, nor will
our generated documentation. All of it assumes that the API is deployed
at the root of the URL. We can change it to use `BASE_URL`, but then
tests will fail if `PATH_PREFIX` is defined unless we switch to `reverse`
retrieve the paths in tests. Doing this, however, creates a ton of
circular imports.
    @sarayourfriend
    sarayourfriend committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    8efdc75 View commit details
    Browse the repository at this point in the history

  2. Start fixing tests; fiddle with circular imports

    @sarayourfriend
    sarayourfriend committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    fe60f71 View commit details
    Browse the repository at this point in the history