Opened 3 years ago

Closed 3 years ago

#33206 closed New feature (wontfix)

Allow redirect type choice in contrib.redirects app

Reported by: Niccolò Mineo Owned by: Niccolò Mineo
Component: contrib.redirects Version: 3.2
Severity: Normal Keywords:
Cc: Adam Johnson Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description (last modified by Niccolò Mineo)

As originally discussed, I would like to propose allowing the user to choose the redirect type (301, 302 and 410 by default) when adding a redirect in the contrib.redirects app admin site. PR is here.

Change History (6)

comment:1 by Niccolò Mineo, 3 years ago

Description: modified (diff)

comment:2 by Niccolò Mineo, 3 years ago

Has patch: set

comment:3 by Niccolò Mineo, 3 years ago

Type: UncategorizedNew feature

comment:4 by Niccolò Mineo, 3 years ago

Description: modified (diff)

comment:5 by Mariusz Felisiak, 3 years ago

Cc: Adam Johnson added
Owner: changed from nobody to Niccolò Mineo
Patch needs improvement: set
Status: newassigned
Triage Stage: UnreviewedAccepted

Thank for this proposition. Tentatively accepted, but only to allow redirect type change (301, 302, 307, 308) per a redirect instance. PR contains unrelated and backward incompatible changes e.g. swapping the redirect model or a new behavior for 410 ("Gone") redirects.

comment:6 by Mariusz Felisiak, 3 years ago

Resolution: wontfix
Status: assignedclosed
Triage Stage: AcceptedUnreviewed

This change is backward incompatible and against our stability policy. We also don't have a clear migration path for existing users. We should go back to the mailing list and reach a strong consensus before moving it forward.

Closing as "wontfix".

Note: See TracTickets for help on using tickets.
Back to Top