fix: allow fragment part in the url - Githubissues

adobe / aem-spa-page-model-manager

Interface between Adobe Experience Manager and Single Page Application framework.

Apache License 2.0

33 stars 24 forks source link

fix: allow fragment part in the url #52

Closed CezCz closed 3 years ago

CezCz commented 3 years ago

Added fragment section to regex as per https://tools.ietf.org/html/rfc3986#section-3.5

(it is the same as query, but starts with # instead of ?)

Related Issue

https://github.com/adobe/aem-spa-page-model-manager/issues/39

How Has This Been Tested?

Unit tested, smoke tested

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.