Summary:

Add an app_name argument to build_integration_urlpatterns().

Review Request #12745 — Created Nov. 28, 2022 and submitted Jan. 8, 2023, 10:19 p.m.

Information
Owner:	chipx86
Repository:	Djblets
Branch:	release-3.x
Bugs:
Depends On:
Reviewers
Groups:	djblets
People:

Description

Django 3.x and higher require specifying an app name when also

specifying a namespace for include() when building URL patterns.

This change adds app_name to complement namespace in

build_integration_urlpatterns().
The function itself now expects keyword arguments. Positional arguments

are supported but deprecated with a warning.
Type hints have also been added throughout djblets.integrations.urls.

Testing Done

Made use of this with RBCommons, which needs this on Django 3.2.

Commits

Summary
	Add an app_name argument to build_integration_urlpatterns().

Screenshots

Issues

Description	From	Last Updated
'django.urls.URLResolver' imported but unused Column: 1 Error code: F401	reviewbot	Nov. 28, 2022, 6:17 p.m.

flake8 failed.

JSHint passed.

flake8

djblets/integrations/urls.py (Diff revision 1)

Show all issues

'django.urls.URLResolver' imported but unused

Column: 1
Error code: F401

Change Summary:

Removed an unused import.

Commits:

	Summary
-		Add an app_name argument to build_integration_urlpatterns().
+		Add an app_name argument to build_integration_urlpatterns().

Diff:

Revision 2 (+102 -10)

Show changes

djblets/integrations/urls.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-3.x (2046b85)