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.

Sign up for GitHub

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

Jump to bottom

Upgrade plotly.js version #7449

Merged
merged 8 commits into from Oct 3, 2023
Merged

Upgrade plotly.js version #7449

merged 8 commits into from Oct 3, 2023

Conversation

sfc-gh-wihuang
Copy link
Contributor

@sfc-gh-wihuang sfc-gh-wihuang commented Sep 28, 2023 •
edited by willhuang1997

Describe your changes

GitHub Issue Link (if applicable)

Closes #7476
Closes #7045

Testing Plan

  • Explanation of why no additional tests are needed
  • Unit Tests (JS and/or Python)
  • E2E Tests
  • Any manual testing needed?

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

@sfc-gh-wihuang sfc-gh-wihuang changed the title Add id to make sure plotly chart actually refreshes Add id to make sure plotly chart actually refreshes with incoming changes Sep 28, 2023
@sfc-gh-wihuang sfc-gh-wihuang changed the title Add id to make sure plotly chart actually refreshes with incoming changes [WIP] Add id to make sure plotly chart actually refreshes with incoming changes Sep 28, 2023
@willhuang1997 willhuang1997 changed the title [WIP] Add id to make sure plotly chart actually refreshes with incoming changes [WIP] Upgrade plotly.js version Oct 2, 2023
@willhuang1997 willhuang1997 marked this pull request as ready for review October 3, 2023 00:02
@willhuang1997 willhuang1997 changed the title [WIP] Upgrade plotly.js version Upgrade plotly.js version Oct 3, 2023
@willhuang1997 willhuang1997 merged commit 0d9d0fc into develop Oct 3, 2023
50 checks passed
@snehankekre snehankekre linked an issue Oct 24, 2023 that may be closed by this pull request
The plotly chart’s “newshape_label” doesn’t show up while using st.plotly_chart function #7594
Closed
4 tasks
@snehankekre snehankekre mentioned this pull request Oct 24, 2023
Closed
4 tasks
@snehankekre snehankekre linked an issue Oct 24, 2023 that may be closed by this pull request
Streamlit doesn't render label text on plotly scatter_mapbox #7553
Closed
4 tasks
@snehankekre snehankekre mentioned this pull request Oct 24, 2023
Closed
4 tasks
eric-skydio pushed a commit to eric-skydio/streamlit that referenced this pull request Dec 20, 2023
@sfc-gh-wihuang @eric-skydio
Upgrade plotly.js version (streamlit#7449)
5c3fb28
zyxue pushed a commit to zyxue/streamlit that referenced this pull request Mar 22, 2024
@sfc-gh-wihuang
Upgrade plotly.js version (streamlit#7449)
a02b2e1
zyxue pushed a commit to zyxue/streamlit that referenced this pull request Apr 16, 2024
@sfc-gh-wihuang @zyxue
Upgrade plotly.js version (streamlit#7449)
382fd74
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kmcgrady kmcgrady kmcgrady approved these changes

Assignees
No one assigned
Labels
change:bugfix impact:users security-assessment-completed
Projects
None yet
Milestone
No milestone
3 participants
@sfc-gh-wihuang @kmcgrady @willhuang1997