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

P UI: Added frontend publishing for bare-metal (git) #5277

Merged
merged 106 commits into from
Jul 20, 2023

Conversation

matmair
Copy link
Contributor

@matmair matmair commented Jul 18, 2023

This PR adds the required tasks and CI for deploying the front end to bare-metal git installs. Code by @wolflu05

Relaces #5275

@matmair matmair requested a review from wolflu05 July 18, 2023 20:37
@wolflu05
Copy link
Contributor

I'll try to review all prs tomorrow.

@matmair matmair marked this pull request as draft July 18, 2023 21:02
@matmair matmair marked this pull request as ready for review July 18, 2023 22:49
@matmair matmair modified the milestones: 1.0.0, 0.13.0 Jul 18, 2023
tasks.py Outdated Show resolved Hide resolved
tasks.py Show resolved Hide resolved
tasks.py Outdated Show resolved Hide resolved
tasks.py Outdated Show resolved Hide resolved
@wolflu05
Copy link
Contributor

Perfect, just only one little thing, the docker-compose run inventree-server invoke update may already be something users documented somewhere on their own updating guides. I would default to --no-frontend if the INVENTREE_DOCKER env is true.

Copy link
Contributor

@wolflu05 wolflu05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SchrodingersGat SchrodingersGat merged commit 2259de7 into inventree:master Jul 20, 2023
15 checks passed
@matmair matmair deleted the wolfu05_bare_metal_ci branch July 20, 2023 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI CI / unit testing ecosystem setup Relates to the InvenTree setup / installation process
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants