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

refactor: make removing index.html from URL configurable #1428

Conversation

mfranzke
Copy link
Contributor

@mfranzke mfranzke commented Feb 10, 2022

Closes #1427

Summary of changes:

I've decided to make this change configurable, especially for backwards "compatibility" (or in other words not changing the current behavior).
Providing the possibility to deactivate the initial redirect and changing of the history towards the removal of an included index.html within the URL: from /index.html?p=WHATEVER to /?p=WHATEVER, as this might cause trouble on environments that don't acknowledge an index.html file if no file is provided in the URL.

Even though that this is a suboptimal configuration, under some circumstances you're able to change it (like on a public static hosting provider).

@mfranzke mfranzke changed the title Refactor/make removing index.html from url configurable refactor: make removing index.html from URL configurable Mar 4, 2022
@JosefBredereck JosefBredereck merged commit 7951a41 into pattern-lab:dev Sep 21, 2022
@JosefBredereck
Copy link
Contributor

PR was released with v5.16.3

@mfranzke mfranzke deleted the refactor/make-removing-index.html-from-url-configurable branch September 25, 2022 13:59
antonia-rose pushed a commit to quelltexterin/nemo-uikit-workshop that referenced this pull request Apr 12, 2023
…#1428)

* refactor: providing the configurability for whether to remove index.html from the URL pattern-lab#1427

* docs(configuration): added entry for noIndexHtmlremoval
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.

No possibility to configure whether or not to remove index.html from the URL when adding to pushState
2 participants