Code

Opened 3 years ago

Closed 3 years ago

#14926 closed (invalid)

Support nesting app namespaces with current_app

Reported by: dcramer Owned by: nobody
Component: Uncategorized Version: 1.2
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

URL resolution should allow nested app namespaces with simple resolution:

reverse('admin:index', current_app='nested:admin')

Right now this isnt possible as the populate method on the resolver doesnt handle recursion. However, explicitly calling applications does work:

reverse('nested:admin:index')

Attachments (0)

Change History (1)

comment:1 Changed 3 years ago by russellm

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to invalid
  • Status changed from new to closed

I'm not sure I follow the advantage here. current_app isn't being provided as a prefix; it's context in which the URL name should be interpreted. For example, there can be multiple admin app namespaces deployed; current_app is used to define which instance of the admin app should be used.

Closing invalid; there isn't enough detail here to establish the API or functionality that is being requested.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.