Skip to content

Remove chart studio upload APIs#7812

Open
marthacryan wants to merge 2 commits into
v4.0from
remove-chart-studio-api
Open

Remove chart studio upload APIs#7812
marthacryan wants to merge 2 commits into
v4.0from
remove-chart-studio-api

Conversation

@marthacryan
Copy link
Copy Markdown
Collaborator

@marthacryan marthacryan commented May 28, 2026

Summary

Removes the APIs that are related to chart studio with the exception of a couple (Plots.sendDataToCloud, plotlyServerURL, the sendDataToCloud modebar button, and the plotly_beforeexport/plotly_afterexport events) in anticipation of adding similar behavior for Plotly Cloud soon.

Removed

Legacy on-graph "Edit chart" link

  • Config options: showLink, linkText, sendData, showSources
  • Plots.addLinks and its positionPlayWithData helper (and the call site in the plot sequence)
  • The showLink = false static-plot override and the showLink passthrough in Snapshot.clone

Legacy streaming

  • The stream trace attribute (stream.token / stream.maxpoints) — vestigial; plotly.js has no streaming client. This was attached to every trace type, hence the bulk of the schema diff.
  • The now-unreachable keepstream branch in Plots.graphJson

"Edit in Chart Studio" pencil button

  • The editInChartStudio modebar button and its showEditInChartStudio config option
  • Simplified manage.js so the cloud button group is driven solely by showSendToCloud

Kept (reusable upload pipeline)

Intentionally left intact for the upcoming Plotly Cloud project:

  • Plots.sendDataToCloud
  • plotlyServerURL config
  • The sendDataToCloud (floppy-disk) modebar button + showSendToCloud config
  • The plotly_beforeexport / plotly_afterexport events

General-purpose export APIs (Plotly.toImage, Plotly.downloadImage, Plotly.Snapshot.*, PlotSchema.get, graphJson) are unaffected.

Testing

  • Updated affected jasmine tests (config_test, modebar_test, snapshot_test)
  • Lint clean; modebar suite passes (103/103). Remaining failures in the local run are pre-existing WebGL/retina environment issues unrelated to this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant