[Django] #28611: Redirects contrib app - CharField(max_length=200) is too little

6 views
Skip to first unread message

Django

unread,
Sep 18, 2017, 5:59:08 AM9/18/17
to django-...@googlegroups.com
#28611: Redirects contrib app - CharField(max_length=200) is too little
---------------------------------------------+---------------------------
Reporter: Artur Barseghyan | Owner: nobody
Type: Uncategorized | Status: new
Component: contrib.redirects | Version: 1.11
Severity: Normal | Keywords: redirects
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
---------------------------------------------+---------------------------
Nowadays, paths are long. It would make sense either:

(Option 1): Change the `old_path` and `new_path` from `CharField` to
`TextField`.
(Option 2): Let the `RedirectFallbackMiddleware` middleware use customized
`Redirect` model.

If any of the solutions is found acceptable, I could come up with a pull
request.

Best regards,

--
Ticket URL: <https://code.djangoproject.com/ticket/28611>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Sep 18, 2017, 8:04:00 AM9/18/17
to django-...@googlegroups.com
#28611: Redirects contrib app - CharField(max_length=200) is too little
-------------------------------------+-------------------------------------

Reporter: Artur Barseghyan | Owner: nobody
Type: | Status: closed
Cleanup/optimization |
Component: contrib.redirects | Version: 1.11
Severity: Normal | Resolution: duplicate

Keywords: redirects | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

* status: new => closed
* type: Uncategorized => Cleanup/optimization
* resolution: => duplicate


Comment:

Duplicate of #19515.

--
Ticket URL: <https://code.djangoproject.com/ticket/28611#comment:1>

Reply all
Reply to author
Forward
0 new messages