Skip to content

Conversation

@iambabyninja
Copy link
Contributor

DASHBOARD_PATH = "/dashboard/"
in .env

@iambabyninja iambabyninja marked this pull request as draft October 2, 2024 15:05
cd `dirname $0`/app/dashboard
VITE_BASE_API=/api/ npm run build --if-present -- --outDir build --assetsDir statics
cp ./build/index.html ./build/404.html
@iambabyninja iambabyninja marked this pull request as ready for review October 2, 2024 15:41
@iambabyninja
Copy link
Contributor Author

Before merging, it's necessary to change the workflow for building the container, specifically the section:

      - name: Build project
        working-directory: ./app/dashboard
        run: VITE_BASE_API=/api/ npm run build --if-present -- --outDir build --base '/dashboard/'

Change it to:

     - name: Build project
        working-directory: ./app/dashboard
        run: VITE_BASE_API=/api/ npm run build --if-present -- --outDir build --assetsDir statics

@ImMohammad20000
Copy link
Contributor

It need to rebuild after chang path?

@ImMohammad20000
Copy link
Contributor

Please change the target branch to dev

@iambabyninja
Copy link
Contributor Author

iambabyninja commented Oct 4, 2024

It need to rebuild after chang path?

no, only restart

@ImMohammad20000
Copy link
Contributor

change the target branch to dev

@iambabyninja
Copy link
Contributor Author

If a merge into the dev branch is required, I will rebuild the frontend and reopen the PR since the versions differ.

@ImMohammad20000
Copy link
Contributor

Yes we need to test it first

@iambabyninja
Copy link
Contributor Author

#1359

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants