-
Notifications
You must be signed in to change notification settings - Fork 481
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
Upgrade react-tooltip #41843
Upgrade react-tooltip #41843
Conversation
The Drone failure here is: learning_platform/level_types/level_group_multi_page_dots again which means maybe you've isolated the issue? |
yes! still confused as to why this upgrade causes the failure, but it's consistent, so i think we've found the root cause. this upgrade isn't essential to getting react upgraded, so i think we can pause on this or have LP revisit separately |
7a7d605
to
cb9dc49
Compare
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.
Wahoo!
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.
Thanks!!
Pulled out of #41582 as it doesn't depend on the React upgrade. Manually verified some tooltips in teacher dashboard after the upgrade.
This upgrade was causing an LP UI test to fail (level_group_multi_page_dots.feature) when inspecting progress in the dropdown. We manually verified the behavior is working as expected and decided to comment those steps out for LP to investigate later (Slack thread / LP Jira follow-up task).