Skip to content

Conversation

@l0uden
Copy link
Contributor

@l0uden l0uden commented Jan 7, 2026

Added hidden to clientside callbacks as configurable parameter.
Now it can be configured from cilentside_callback function.

Solves this issue: #3525

Contributor Checklist

  • I have run the tests locally and they passed. (refer to testing section in contributing)

T4rk1n

This comment was marked as outdated.

None,
prevent_initial_call,
no_output=no_output,
hidden=True,
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't quite understand why it is mandatory to set it to True here.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh right, it should be taken from the args, not hide all the clientside callbacks

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks!

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.

Should take the hidden parameter from the arguments.

None,
prevent_initial_call,
no_output=no_output,
hidden=True,
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh right, it should be taken from the args, not hide all the clientside callbacks

@l0uden l0uden changed the title Add hidden=True to clientside callbacks Add hidden to clientside callbacks as configurable parameter Jan 8, 2026
@l0uden l0uden requested review from CNFeffery and T4rk1n January 8, 2026 17:19
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.

3 participants