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

Code breaks on latest release #581

Closed
HarryKodden opened this issue Apr 22, 2020 · 5 comments
Closed

Code breaks on latest release #581

HarryKodden opened this issue Apr 22, 2020 · 5 comments

Comments

@HarryKodden
Copy link

@HarryKodden HarryKodden commented Apr 22, 2020

Actual Behavior

from wtforms.widgets import HTMLString
ImportError: cannot import name 'HTMLString'

Environment

  • Python version: 3.6.8
  • wtforms version: 2.3.0
@spookey
Copy link

@spookey spookey commented Apr 22, 2020

I had the same issue when updating.
HTMLString was replaced with Markup. So use from markupsafe import Markup instead.

Also from wtforms.widgets.core import escape_html was replaced by from markupsafe import escape

@R11baka
Copy link

@R11baka R11baka commented Apr 22, 2020

I had the same issue when running airflow

@hatched-DavidMichon
Copy link

@hatched-DavidMichon hatched-DavidMichon commented Apr 22, 2020

Also got this issue

@kaxil
Copy link

@kaxil kaxil commented Apr 22, 2020

Is it possible to restore HTMLString and cut 2.3.1 since it specifically said it was a backwards incompatible change in the #400?

@davidism
Copy link
Member

@davidism davidism commented Apr 22, 2020

Fixed in #583.

kaxil added a commit to astronomer/airflow that referenced this issue Jun 23, 2020
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581
kaxil added a commit to astronomer/airflow that referenced this issue Jun 23, 2020
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581
kaxil added a commit to astronomer/airflow that referenced this issue Jun 23, 2020
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581
kaxil added a commit to apache/airflow that referenced this issue Jun 23, 2020
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581
kaxil added a commit to kaxil/airflow that referenced this issue Jun 27, 2020
…9487)

WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Sep 15, 2021
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581

GitOrigin-RevId: 7c587dc486c4f637c681e2e8b61a12ebf99c0f14
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Sep 17, 2021
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581

GitOrigin-RevId: 7c587dc486c4f637c681e2e8b61a12ebf99c0f14
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Sep 23, 2021
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581

GitOrigin-RevId: 7c587dc486c4f637c681e2e8b61a12ebf99c0f14
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Nov 25, 2021
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581

GitOrigin-RevId: 7c587dc486c4f637c681e2e8b61a12ebf99c0f14
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Mar 9, 2022
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581

GitOrigin-RevId: 7c587dc486c4f637c681e2e8b61a12ebf99c0f14
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Jun 3, 2022
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581

GitOrigin-RevId: 7c587dc486c4f637c681e2e8b61a12ebf99c0f14
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Jun 6, 2022
WTForms uses `MarkUp` to escape strings now and removed their internal class HTMLString in Master. Details: wtforms/wtforms#400

That change previously broke Airflow for new users (in 2.3.0). However on users request they added `HTMLString` that just passes all args to `markupsafe.MarkUp` back for temporary Backward compatbility with deprecation warning in 2.3.1. Details: wtforms/wtforms#581

GitOrigin-RevId: 7c587dc486c4f637c681e2e8b61a12ebf99c0f14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

6 participants