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

Fix string extraction from nullable RichTextField #621

Merged
merged 1 commit into from
Sep 23, 2022

Conversation

zerolab
Copy link
Collaborator

@zerolab zerolab commented Sep 23, 2022

Fixes #487, and replaces #523

Kudos to @vladox and @Timusan

@zerolab zerolab added this to the 1.3 milestone Sep 23, 2022
@zerolab zerolab force-pushed the fix/extract-strings-with-no-value branch from 7e6de4d to e37726b Compare September 23, 2022 14:33
@zerolab zerolab force-pushed the fix/extract-strings-with-no-value branch from e37726b to 96b0107 Compare September 23, 2022 14:41
@@ -1,4 +1,4 @@
# Generated by Django 4.0.5 on 2022-06-05 11:41
# Generated by Django 4.1.1 on 2022-09-23 14:13
Copy link
Contributor

Choose a reason for hiding this comment

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

This isn't a problem with this PR (if it was actually causing a problem, a test would have failed), but just while I am reading this line, a quick note.

For django packages, the recommended practice is to always generate your migrations with the lowest version of django your package is compatible with (which in this case would be 3.2). See https://docs.djangoproject.com/en/4.1/topics/migrations/#supporting-multiple-django-versions for explanation.

Copy link
Contributor

Choose a reason for hiding this comment

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

(this matters even less if the migration is for the test app 😄 )

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it doesn't matter as I've made the necessary modifications for the initial migration manually

@zerolab zerolab merged commit 821e1f0 into main Sep 23, 2022
@zerolab zerolab deleted the fix/extract-strings-with-no-value branch September 23, 2022 16:04
@vladox
Copy link

vladox commented Sep 28, 2022

@zerolab great job thxs!

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.

Error while translating: TypeError: object of type 'NoneType' has no len()
3 participants