tethysplatform / tethys / 18735124200

100%

push

github

SINGLE_APP_MODE Url Fix with React App (#1204)

* for MULTIPLE_APP_MODE = false, move the tethys_app urls to the end to not override other django urls

* move all app urls to the end of the url patterns

* added prefix to MEDIA_URL so that it included the prefix url on python import

update urls for MEDIA_URL to create the urls with re.path so that MEDIA_URL setting could have the prefix

* updated webpack template with prefix url fix

* Updated templates to account for prefix_url

* updated scaffold command to take into consideration the prefix_url

* more tests and using settings.MEDIA_URL for the media path instead of a hardcoded "media"

* fix typo on prefix type

* updated changes for nginx on gen command

* reverted some code to use django static function for media urls. This was so that django serves static media data only in DEBUG false mode

* updated test for nginx and apache taking into account the prefix_url

* removed hardcoded tethysdash reference in webpack template for react app
fixed issue in getTethysPortalHost where the result url had double quotes when the prefix url is empty

* updated apache to work with prefix

* black formatted

* fix error on test with prefix

* fixed test case for nginx

* removed tethysdash specific function
removed prefix from getTethysPortalHost

* removed prefix_url from scaffold and react app environment templates
removed '/apps/' from Error Exit app because '/apps/' probably doesnt exist in single app mode

* fixed flake8

* taking into account single app mode

* need prefix function

* update getTethysPortalHostWithPrefix  to be getTethysPortalBase

* updated getTethysPortalBase to work correctly if there is no prefix

* fixed "Exit the App" link on the error page to handle a prefix

* updated env comments for single app mode

---------

Co-authored-by: romer8 <gio.rombus@gmail.com>

9 of 9 new or added lines in 2 files covered. (100.0%)

11886 of 11886 relevant lines covered (100.0%)

1.0 hits per line