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

Remove static url configuration that improperly sets url path to a fi… #206

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

christabor
Copy link
Owner

…le path, resulting in a 500 error in Windows. Closes #128.

Locally I'm able to confirm this works both during testing and in manual viewing of the page. I think this was originally set for the purpose of making it installable not just as an app, but as an imported package. However, the docs for flask indicate that this is always relative to the application, and this is configured to use an absolute path anyway:

STATIC_DIR = os.path.dirname(static.__file__)

so, this should negate the need to manually set both.

screen shot 2019-02-28 at 9 55 36 pm

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.

1 participant