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

Inconsistency path for static files #4451

Closed
1 of 7 tasks
Vultik opened this issue Mar 4, 2023 · 3 comments · Fixed by inventree/inventree-docs#449
Closed
1 of 7 tasks

Inconsistency path for static files #4451

Vultik opened this issue Mar 4, 2023 · 3 comments · Fixed by inventree/inventree-docs#449
Labels
setup Relates to the InvenTree setup / installation process

Comments

@Vultik
Copy link

Vultik commented Mar 4, 2023

Deployment Method

  • Installer
  • Docker Development
  • Docker Production
  • Bare metal Development
  • Bare metal Production
  • Digital Ocean image
  • Other (please provide a link Steps to Reproduce

Describe the problem

In the docs, it's written to create a folder for static files at /home/inventree/static . However, in the config.yaml the default path is

image

If you uncomment theses lines, it will not match the paths suggested in the doc.
So when you run the invoke update command, the generates files will be stored at /home/inventree/data/static instead of /home/inventree/static.

Suggested fix : change the default static path in the config_template.yml file to /home/inventree/static

Steps to Reproduce

https://docs.inventree.org/en/latest/start/install/#bare-metal-setup

Relevant log output

No response

@Vultik Vultik added question This is a question setup Relates to the InvenTree setup / installation process triage:not-checked Item was not checked by the core team labels Mar 4, 2023
@matmair matmair removed question This is a question triage:not-checked Item was not checked by the core team labels Mar 4, 2023
@matmair
Copy link
Member

matmair commented Mar 4, 2023

@Vultik our docker containers use the data subdir to make mounting folders cleaner - that is where the path comes from. Docker deployment seems to be used more by new users than bare metal - which gives you the most options but also requires deeper linux knowledge than a package or docker install.

I think it can be expected from users to read what they put into their config file. @SchrodingersGat you d'accor?

@SchrodingersGat
Copy link
Member

@Vultik the bare-metal docs provide a starting point but the directories should be located based on your requirements.

That being said, there is definitely some room for improvement in the docs. As @matmair points out, the docker setup requires a particular directory structure. But the installer configures the directories differently.

Potentially the bare-metal setup guide should be closer to the documentation for our bare-metal installer...

@Vultik
Copy link
Author

Vultik commented Mar 5, 2023

The updated docs are more clearer now .
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
setup Relates to the InvenTree setup / installation process
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants