Opened 21 months ago

Closed 21 months ago

Last modified 21 months ago

#31270 closed Cleanup/optimization (fixed)

Documentation for RedirectView.get_redirect_url() doesn't describe its arguments.

Reported by: Baptiste Mispelon Owned by: Hasan Ramezani
Component: Documentation Version: 3.0
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

The documentation for RedirectView.get_redirect_url() [1] shows that the method takes *args, **kwargs as arguments but doesn't explain what those are exactly.

I had to look at the code to be sure: args and kwargs are the groups captured in the URL. That's what I expected and it might be obvious but I still think it should be mentioned in this reference documentation.

I don't have immediate plans to work on this, anyone can pick this up. The file to modify is in docs/ref/class-based-views/base.txt [2].

[1] https://docs.djangoproject.com/en/dev/ref/class-based-views/base/#django.views.generic.base.RedirectView.get_redirect_url
[2] https://github.com/django/django/blob/master/docs/ref/class-based-views/base.txt#L259

Change History (7)

comment:1 Changed 21 months ago by Mariusz Felisiak

Summary: Documentation for RedirectView.get_redirect_url doesn't describe its argumentsDocumentation for RedirectView.get_redirect_url doesn't describe its arguments.
Triage Stage: UnreviewedAccepted

comment:2 Changed 21 months ago by Mariusz Felisiak

Summary: Documentation for RedirectView.get_redirect_url doesn't describe its arguments.Documentation for RedirectView.get_redirect_url() doesn't describe its arguments.

comment:3 Changed 21 months ago by Hasan Ramezani

Owner: changed from nobody to Hasan Ramezani
Status: newassigned

comment:4 Changed 21 months ago by Hasan Ramezani

Has patch: set

comment:5 Changed 21 months ago by Mariusz Felisiak

Triage Stage: AcceptedReady for checkin

comment:6 Changed 21 months ago by Mariusz Felisiak <felisiak.mariusz@…>

Resolution: fixed
Status: assignedclosed

In 2ab97af:

Fixed #31270 -- Doc'd RedirectView.get_redirect_url() arguments.

comment:7 Changed 21 months ago by Mariusz Felisiak <felisiak.mariusz@…>

In bcf58e3e:

[3.0.x] Fixed #31270 -- Doc'd RedirectView.get_redirect_url() arguments.

Backport of 2ab97af3528352e8f3ea4aa725b863822442d5ed from master

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