-
-
Notifications
You must be signed in to change notification settings - Fork 31.1k
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
Bootstrapped Django 5.0. #16432
Bootstrapped Django 5.0. #16432
Conversation
A couple of days ago, I noticed that the 4.0 release notes don't include the "Third-party library support for older version of Django" section that's in the 3.0 release notes. It probably doesn't need to be added there at this point, but it could be useful to add it to 5.0. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think everything is fine
Thanks 👍 Added. |
I'm going to continue next week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you made a good job here, really.
While reviewing, I noticed we have a serious problem with Form.localize/Widget.is_localized attributes which are basically not doing what they should as per the documentation. They are mostly initialized as False
while the values are still localized. But of course, working on this issue is not the objective of this PR.
These changes also show that the various date format settings in global settings will be hardly usable in practice (unless some work is done on Field.localize, defaulting to True and make False value have real effects).
@claudep Thanks for the review ⭐ |
1abce32
to
de74147
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @felixxm — looks really good.
My only real comment is the zoneinfo
being universally available now, so the imports can be simplified and the backports.zoneinfo
dependencies removed. (I can pick this up first thing Tuesday if you want to collapse on it 🙂)
Thanks so much for the sterling effort here! 🎁
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@felixxm I'm approving modulo the previous comment (#16432 (comment)) in case you want to push on on Monday!
…ict/ErrorList. Per deprecation timeline.
…precation timeline.
…ed_property(). Per deprecation timeline.
…ase.assertFormError()/assertFormsetErrors(). Per deprecation timeline.
…eError after prefetch_related(). Per deprecation timeline.
…ated filters. Per deprecation timeline.
…ackend.configure_user() subclasses. Per deprecation timeline.
Per deprecation timeline.
…ormset name to SimpleTestCase.assertFormError()/assertFormSetError(). Per deprecation timeline.
…deprecation timeline.
…r per deprecation timeline.
…v-based. Per deprecation timeline. This also removes "django/forms/default.html" and "django/forms/formsets/default.html" templates.
…ror when nulls_first/nulls_last=False is passed. Per deprecation timeline.
:mod:`django.contrib.admindocs` | ||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||
|
||
* The default iteration count for the PBKDF2 password hasher is increased from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This belongs in the auth section, not admindocs. ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(to be merged after the
stable/5.0.x
branch is created)