Changes between Initial Version and Version 1 of Ticket #22425, comment 11
- Timestamp:
- May 7, 2021, 12:16:30 PM (4 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #22425, comment 11
initial v1 1 1 > What do you think about it? 2 2 3 Instead of having two very similar arguments, I would justsuggest having one. What about the following (similar to what I suggested above)?3 Instead of having two very similar arguments, I would suggest having one. What about the following (similar to what I suggested above)? 4 4 5 5 {{{#!python … … 7 7 }}} 8 8 9 The `catch_all` argument can be either a boolean or a view. If true, it aborts early and uses the default catch-all view (e.g. the debug 404 page when debug is enabled). Otherwise, it would use the given view. I also think the view argument should just be a view (like what the related [https://docs.djangoproject.com/en/3.2/ref/urls/#django.urls.path path()] function accepts), rather than a string. I would need to look at the code more closely, but there's a chance a third option might be useful.9 The `catch_all` argument (read as "catch all URL patterns") can be either a boolean or a view. If true, it aborts early and uses the default catch-all view (e.g. the debug 404 page when debug is enabled). Otherwise, it would use the given view. I also think the view argument should just be a view (like what the related [https://docs.djangoproject.com/en/3.2/ref/urls/#django.urls.path path()] function accepts), rather than a string. I would need to look at the code more closely, but there's a chance a third option might be useful.