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

feat: Add Redirect map #1181

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

ArneGudermann
Copy link
Contributor

Resolves #1168

This PR adds an Redirect handler with two modes.
In the normal mode its only matches the key in the dict.

In the advanced mode you can provides key with wildcards like:

conf.redirect_map["/test/add*"]="/test/edit/"

@ArneGudermann ArneGudermann added feature New feature or request Priority: Low This issue can be considered with enough idle time. labels Jun 7, 2024
@ArneGudermann ArneGudermann added this to the ViUR-core v3.7 milestone Jun 7, 2024
@ArneGudermann ArneGudermann linked an issue Jun 7, 2024 that may be closed by this pull request
@phorward phorward added viur-meeting Issues to discuss in the next ViUR meeting and removed viur-meeting Issues to discuss in the next ViUR meeting labels Jun 21, 2024
src/viur/core/request.py Outdated Show resolved Hide resolved
sveneberth
sveneberth previously approved these changes Oct 22, 2024
src/viur/core/config.py Outdated Show resolved Hide resolved
src/viur/core/config.py Outdated Show resolved Hide resolved
Copy link
Member

@phorward phorward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ArneGudermann, I would like to combine this pull request with #1218 (#803) to get rid of the SEO-stuff. Your redirect-map is part of it, but should come from a SkelModule.

If this PR is not necessary for current customer projects, I would like to shift it to v3.8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request Priority: Low This issue can be considered with enough idle time.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

a Simple Redirect Handler for mapping urls
3 participants