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

redirect with view #132

Merged
merged 4 commits into from
Mar 24, 2022
Merged

redirect with view #132

merged 4 commits into from
Mar 24, 2022

Conversation

mamico
Copy link
Member

@mamico mamico commented Feb 14, 2022

This is a #130 follow-up.

The previous pull request fixes the redirect for content, but doesn't work for views (my fault, sorry).

p.s I'm opening another PR about changing from 301 to safer 302 redirects, as in p.a.redirector (refs. plone/plone.app.redirector#8 plone/plone.app.redirector#22)

@mister-roboto

This comment was marked as resolved.

@mamico mamico requested a review from jensens February 14, 2022 11:15
@mamico
Copy link
Member Author

mamico commented Feb 14, 2022

@jenkins-plone-org please run jobs

1 similar comment
@mamico
Copy link
Member Author

mamico commented Mar 8, 2022

@jenkins-plone-org please run jobs

@tisto tisto merged commit 4891ad1 into master Mar 24, 2022
@tisto tisto deleted the fix_redirector_with_new_traverser_views branch March 24, 2022 15:57
tisto added a commit that referenced this pull request Mar 24, 2022
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.

4 participants