Skip to content

Add support for Django 4#24

Merged
sonus21 merged 2 commits intosonus21:masterfrom
jeduardo211:django_4
May 26, 2022
Merged

Add support for Django 4#24
sonus21 merged 2 commits intosonus21:masterfrom
jeduardo211:django_4

Conversation

@jeduardo211
Copy link
Copy Markdown
Contributor

Django 4.0 no longer contains the url method for defining routes.
This pull request defines the routers using the more modern path method, and defines the app_name to allow for URLs namespaces.

The old way of defining urls in DJango is no longer supported in Django 4.0.

This commit replaces the regular expression based paths by the more modern path method, and also adds app_name in the urls file to allow usage with URL namespaces.
Altered major version because old versions of Django cannot be supported anymore.
@denpufa
Copy link
Copy Markdown

denpufa commented May 9, 2022

can aprove this pr?

@sonus21
Copy link
Copy Markdown
Owner

sonus21 commented May 10, 2022

@denpufa can you please share test report?
./run-tests.sh

@sonus21
Copy link
Copy Markdown
Owner

sonus21 commented May 10, 2022

@jeduardo211 ^

@denpufa
Copy link
Copy Markdown

denpufa commented May 25, 2022

/home/rodolfo/error-tracker/error_tracker/django/__init__.py:28: UserWarning: APP_ERROR_DB_MODEL is not set using default model
  warnings.warn("APP_ERROR_DB_MODEL is not set using default model")
/home/rodolfo/error-tracker/error_tracker/libs/utils.py:121: UserWarning: Default Masking module will be used
  warnings.warn(message)
/home/rodolfo/error-tracker/error_tracker/libs/utils.py:121: UserWarning: Default ContextBuilder module will be used
  warnings.warn(message)
Creating test database for alias 'default'...
/home/rodolfo/error-tracker/error_tracker/django/__init__.py:28: UserWarning: APP_ERROR_DB_MODEL is not set using default model
  warnings.warn("APP_ERROR_DB_MODEL is not set using default model")
/home/rodolfo/error-tracker/error_tracker/django/__init__.py:28: UserWarning: APP_ERROR_DB_MODEL is not set using default model
  warnings.warn("APP_ERROR_DB_MODEL is not set using default model")
.................
----------------------------------------------------------------------
Ran 17 tests in 1.685s

OK

@sonus21 django report here, want flask too?

@sonus21 sonus21 merged commit d819f97 into sonus21:master May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants