티스토리 수익 글 보기

티스토리 수익 글 보기

Bootstrapped Django 5.0. by felixxm · Pull Request #16432 · django/django · GitHub
Skip to content

Conversation

felixxm
Copy link
Member

@felixxm felixxm commented Jan 6, 2023

(to be merged after the stable/5.0.x branch is created)

@timgraham
Copy link
Member

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.

Copy link

@emarhin emarhin left a 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

@felixxm
Copy link
Member Author

felixxm commented Jan 6, 2023

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.

Thanks 👍 Added.

@felixxm
Copy link
Member Author

felixxm commented Jan 6, 2023

I’m going to continue next week.

Copy link
Member

@claudep claudep left a 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).

@felixxm
Copy link
Member Author

felixxm commented Jan 7, 2023

@claudep Thanks for the review ⭐

@felixxm felixxm force-pushed the django50 branch 10 times, most recently from 1abce32 to de74147 Compare January 13, 2023 09:30
@felixxm felixxm changed the title [WiP] Bootstrapped Django 5.0. Bootstrapped Django 5.0. Jan 13, 2023
@felixxm felixxm marked this pull request as ready for review January 13, 2023 10:00
@felixxm felixxm requested a review from carltongibson January 13, 2023 13:05
Copy link
Member

@carltongibson carltongibson left a 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! 🎁

Copy link
Member

@carltongibson carltongibson left a 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!

…ase.assertFormError()/assertFormsetErrors().

Per deprecation timeline.
…eError after prefetch_related().

Per deprecation timeline.
…ackend.configure_user() subclasses.

Per deprecation timeline.
…ormset name to SimpleTestCase.assertFormError()/assertFormSetError().

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.
@felixxm felixxm merged commit 4fc711a into django:main Jan 17, 2023
@felixxm felixxm deleted the django50 branch January 17, 2023 10:49
:mod:`django.contrib.admindocs`
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

* The default iteration count for the PBKDF2 password hasher is increased from
Copy link
Member

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. 😉

Copy link
Member Author

Choose a reason for hiding this comment

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

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.

6 participants