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

900 context path #926

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

900 context path #926

wants to merge 26 commits into from

Conversation

longshuicy
Copy link
Member

@longshuicy longshuicy commented Feb 12, 2024

Update:
Everything should work now. For testing please refer to below section.


Test development:

  1. export BASE_URL_ROUTE={your subdirectory}
  2. npm run start:dev
  3. go to http://localhost:3000/{your subdirectory}/. Any navigation between links should automatically goes to /subdirectory
  4. unset BASE_URL_ROUTE && npm run start:dev. Everything should still run under root normally.
    Note: when logging in, the redirect_uri need to be configured in keycloak to reflect that subdirectory

Test production:

Please follow the instruction documentation. This includes several steps: e.g. Build frontend image; Run specific docker compose with context path; Configure keycloak

Make sure to test backward compatibility as well:

  1. docker compose up
  2. Everything should still run under root normally.

@longshuicy longshuicy requested a review from max-zilla February 12, 2024 18:57
@longshuicy longshuicy linked an issue Feb 12, 2024 that may be closed by this pull request
@longshuicy
Copy link
Member Author

@max-zilla I recall you made some progress deploying under a subroute.
If possible, could you share your branch/ example where you've been experimenting with?

@longshuicy longshuicy marked this pull request as ready for review August 16, 2024 15:49
@longshuicy longshuicy requested a review from lmarini as a code owner August 16, 2024 15:49
@longshuicy longshuicy marked this pull request as draft August 16, 2024 19:27
@longshuicy longshuicy marked this pull request as ready for review August 19, 2024 23:14
@longshuicy longshuicy requested review from tcnichol and ddey2 August 19, 2024 23:14
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.

Context path
1 participant