Skip to content

BASE_URL support #248

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

BASE_URL support #248

wants to merge 2 commits into from

Conversation

cyrus28214
Copy link

Enable subpath deployment via BASE_URL for easier reverse proxy/subdirectory hosting. Close #198.

Changes:

  1. vite.config.ts: set base from BASE_URL
  2. Router: BrowserRouter basename={import.meta.env.BASE_URL}
  3. i18n: loadPath prefixed by BASE_URL
  4. Assets: background via imports; fonts imported in src/index.tsx

Usage:

export BASE_URL='/omni-tools/'
npm run dev
# or
npm run build
npm run serve

Help Needed:

  1. Can anyone help me check if PWA and Docker works well with BASE_URL?
  2. I am not sure if I've caught all the absolute paths.

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

@iib0011 iib0011 Awaiting requested review from iib0011 iib0011 is a code owner

@Chesterkxng Chesterkxng Awaiting requested review from Chesterkxng Chesterkxng is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

BASE_URL Support
1 participant
@cyrus28214