show namespaces alongside url_name in request panel - #1442
Conversation
|
Thanks, this is great. I think including namespaces directly in the URL name is fine. However, the new test seems to fail quite often. @AlexCLeduc Do you have an idea why? |
Codecov Report
@@ Coverage Diff @@
## master #1442 +/- ##
==========================================
+ Coverage 87.44% 88.01% +0.56%
==========================================
Files 29 29
Lines 1577 1593 +16
Branches 220 223 +3
==========================================
+ Hits 1379 1402 +23
+ Misses 146 141 -5
+ Partials 52 50 -2
Continue to review full report at Codecov.
|
|
@matthiask Sorry about that, I had trouble running tests locally and figured CI was misconfigured. Turns out the test app never included the admin URLs, looks like it works now. |
|
@AlexCLeduc If possible, could you move this test to the |
eb30b0f to
39bc38b
Compare
|
Thank you! 💪🎉 |
As far as I know, no one really asked for this, but I work on projects that make heavy use of namespaces with repetitive urls (e.g. create, edit, index) and having namespaces available in the request panel helps disambiguate and makes for a quicker grep.
Before this PR:

After this PR:

I used
:as a separator because that's what you'd pass to e.g.reverse("admin:login"). We can alternatively add a separate 'namespace' title/content section, but this seemed cleaner to me