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

Docs: Updates to History Mode for Apache w/ subfolder #3066

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

itsxallwater
Copy link

Reason
I had issues getting this to work with the examples in the doc as currently written. It wasn't until I found this answer on Stack Overflow where things became clear. Admittedly, I'm not well versed in Apache, but I figured these amendments might better serve others in the future. Per the Stack Overflow answer it seems there was a related Vue Forum discussion which tells me the scenario is not uncommon.

What changed?
I added a few specifics to the Apache example in history-mode.md and removed the trailing / in the RewriteBase example for subfolders. These adjustments mirror how I was able to get my application working when building to a different base path and deploying to Apache.

As-is, the docs for how to configure Apache when using a sub-directory were not sufficient for me. I was not able to get it to work until I expanded the configuration a la the suggested doc updates.
Changed verbiage and placement to more closely align with the Apache example.
@posva posva added the docs label Dec 17, 2019
@posva posva added the need review Needs to be reviewed by someone who know about the topic label Aug 3, 2020
@chenkunlong
Copy link

sd

@Alanscut
Copy link
Contributor

Hi @itsxallwater , could you add apacheconf after ``` to highlight it?

Change apache to apacheconf
@itsxallwater
Copy link
Author

Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs need review Needs to be reviewed by someone who know about the topic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants