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 )
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 , 3 years ago
Description: | modified (diff) |
---|
comment:2 by , 3 years ago
Has patch: | set |
---|
comment:3 by , 3 years ago
Type: | Uncategorized → New feature |
---|
comment:4 by , 3 years ago
Description: | modified (diff) |
---|
comment:5 by , 3 years ago
Cc: | added |
---|---|
Owner: | changed from | to
Patch needs improvement: | set |
Status: | new → assigned |
Triage Stage: | Unreviewed → Accepted |
comment:6 by , 3 years ago
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
Triage Stage: | Accepted → Unreviewed |
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.
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.