Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update urls.py #92

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update urls.py #92

wants to merge 2 commits into from

Conversation

cr0hn
Copy link

@cr0hn cr0hn commented Nov 3, 2022

Add compatibility with Django 4.0, maintaining the compatibility for older Django versions

Add compatibility with Django 4.0, maintaining the compatibility for older Django versions
@midupe
Copy link

midupe commented Dec 19, 2022

please update urls.py in pip install django-pwa

re_path(r'^manifest\.json$', manifest, name='manifest'),
re_path('^offline/$', offline, name='offline')
]
if VERSION >= (4, 0):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

path exists since Django 2.0. I don't think this condition is necessary.

I do think it would be more helpful for the project to drop support for unsupported Django versions.
We should update setup.py to Django 2.0+ (or even 3.2+).

@hartungstenio
Copy link
Collaborator

@cr0hn could you please add the missing Django versions to the classifiers?

Added support for Django 4 and 4.1. Add Python 3.9 - 3.10
@cr0hn
Copy link
Author

cr0hn commented Dec 21, 2022

Done

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