Skip to content

Support processUnpinnedColumns#470

Open
datenzauberai wants to merge 7 commits into
plotly:v35from
datenzauberai:support_processUnpinnedColumns
Open

Support processUnpinnedColumns#470
datenzauberai wants to merge 7 commits into
plotly:v35from
datenzauberai:support_processUnpinnedColumns

Conversation

@datenzauberai

@datenzauberai datenzauberai commented Jun 27, 2026

Copy link
Copy Markdown

This would close #468

It basically just adds processUnpinnedColumns to src/lib/utils/propCategories.js and adds a test for this.

Please feel free to make changes.

Comment thread tests/utils.py
Comment thread tests/utils.py
Comment thread tests/test_column_pinning.py
@AnnMarieW

Copy link
Copy Markdown
Collaborator

Hi @datenzauberai

Thanks so much for the PR and the updated tests 🎉

I'm going to switch this to the v35 branch, which should fix the unrelated tests that are failing.
What version are you using? Do you need this applied to an earlier version?

@AnnMarieW AnnMarieW changed the base branch from main to v35 June 27, 2026 15:32
@datenzauberai

datenzauberai commented Jun 28, 2026

Copy link
Copy Markdown
Author

I'm going to switch this to the v35 branch, which should fix the unrelated tests that are failing.

Perfect.

What version are you using? Do you need this applied to an earlier version?

We're on 33, but we're not blocked by this, because we workaround using a clientside callback to set the parameter. Hence, any release would just be super fine for us.

Comment thread tests/test_column_pinning.py
@BSd3v

BSd3v commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

@datenzauberai

We need a changelog entry.

@sonarqubecloud

sonarqubecloud Bot commented Jul 2, 2026

Copy link
Copy Markdown

@datenzauberai

Copy link
Copy Markdown
Author

@datenzauberai

We need a changelog entry.

Done.

@BSd3v BSd3v left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

💃

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.

Support processUnpinnedColumns as a function

3 participants