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(dashboard)!: Downgrade WTForms to >=2.3.3, <3.1.0 due to compatibility issues #25643

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wanjoc
Copy link

@wanjoc wanjoc commented Oct 14, 2023

SUMMARY

While installing Superset via pip, WTForms version 3.1.0 is fetched, which is incompatible. Pinning the version to 3.0.1 resolves errors on pages like 'List Users', 'Roles', and 'Activity'.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screen Shot 2023-10-14 at 11 46 42 AM Screen Shot 2023-10-14 at 11 50 05 AM

Please refer to the following issues for more screenshots and further details:
Fixes #25622
Fixes #25631

TESTING INSTRUCTIONS

  1. Setup Superset locally using these instructions.
  2. Start the development web server on port 8088.
  3. Visit http://localhost:8088/users/list/). The aforementioned error should be present when you check the logs in terminal.
  4. Uninstall WTForms (pip uninstall wtforms) and reinstall the compatible version (pip install wtforms==3.0.1). The error should no longer be present when you navigate to the /users endpoint

Additional Observation:
Post this update, a UI issue was observed when I ran pip install . instead of pip install apache-superset as per the documentation. However, this seems unrelated to the changes in setup.py since it persists even when reverting to the original state and running the same command. Any insights into this would be appreciated.

ADDITIONAL INFORMATION

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Congrats on making your first PR and thank you for contributing to Superset! 🎉 ❤️

We hope to see you in our Slack community too! Not signed up? Use our Slack App to self-register.

@@ -130,7 +130,7 @@ def get_git_sha() -> str:
"typing-extensions>=4, <5",
"waitress; sys_platform == 'win32'",
"werkzeug>=2.3.3, <3",
"wtforms>=2.3.3, <4",
Copy link
Member

Choose a reason for hiding this comment

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

This was fixed on FAB and bumped here: #25712

Copy link
Author

Choose a reason for hiding this comment

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

Confirmed. Would it be appropriate for me to close this PR now?

Copy link
Member

Choose a reason for hiding this comment

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

I just posted on #25631 asking them to confirm that the issue does not persist in Superset 3.0.2, which uses a newer version of FAB. Especially if someone can confirm that this issue is solved, then I would think this PR can be closed.

Copy link
Member

Choose a reason for hiding this comment

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

I closed the other issue because of a lack of response. I'm guessing it's safe now to close this issue, but let me know if anyone disagrees!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to open user list Superset 3.0 pypi package installs wrong version of wtforms
4 participants