Skip to content

Conversation

BSd3v
Copy link
Contributor

@BSd3v BSd3v commented Sep 1, 2025

This brings patch functionality to:

  • set_props both clientside and server side
  • exposed a PatchBuilder class at 'Patch' on the clientside to build the user friendly version of the patch

fixes #3101

Copy link
Contributor

@T4rk1n T4rk1n left a comment

Choose a reason for hiding this comment

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

Looks good, just need a changelog entry and tests similar to the one we have for Python side.

Copy link
Contributor

@T4rk1n T4rk1n left a comment

Choose a reason for hiding this comment

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

💃

@gvwilson gvwilson added P1 needed for current cycle fix fixes something broken community community contribution labels Sep 9, 2025
@gvwilson
Copy link
Contributor

gvwilson commented Sep 9, 2025

@T4rk1n please merge when you're ready

@T4rk1n T4rk1n merged commit 4b8917e into plotly:dev Sep 9, 2025
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community community contribution fix fixes something broken P1 needed for current cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

set_props does not work with Patch()
3 participants