-
-
Notifications
You must be signed in to change notification settings - Fork 173
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
[IMP]convert_field_to_html: manage translated fields #384
[IMP]convert_field_to_html: manage translated fields #384
Conversation
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.
LGTM (code review)
@pedrobaeza @legalsylvain maybe this interest you. It replace the following PR : #358 |
openupgradelib/openupgrade.py
Outdated
""" | ||
Convert field value to HTML value. | ||
|
||
.. versionadded:: 7.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.
Don't add empty lines inside a method, please (and the same for the next one)
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.
The empty lines still exist
8c378b9
to
8a5d4be
Compare
What are the next step to merge and create a new release usable with pip ? |
8a5d4be
to
0206473
Compare
0206473
to
b7c7ec9
Compare
Improve convert_field_to_html method: added translate=False parameter to handle translatable fields