Skip to content

shared-signals.adoc: Replace ValueSignal with SharedValueSignal#5635

Open
DarioViva42 wants to merge 1 commit into
vaadin:v25.1from
DarioViva42:patch-2
Open

shared-signals.adoc: Replace ValueSignal with SharedValueSignal#5635
DarioViva42 wants to merge 1 commit into
vaadin:v25.1from
DarioViva42:patch-2

Conversation

@DarioViva42
Copy link
Copy Markdown
Contributor

This documentation describes the usage of Shared Signals. So it is really confusing and probably a mistake, that in some of the examples Local Signals are used.

This documentation describes the usage of Shared Signals. So it is really confusing and probably a mistake, that in some of the examples Local Signals are used.
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Jun 5, 2026

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Jun 5, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@DarioViva42
Copy link
Copy Markdown
Contributor Author

Also what I also find pretty Strange that there is a Shared Signal SharedNumberSignal but not a Local Signal NumberSignal. There are also similar discrepancies in this regard. It's especially strange because in an earlier edition of Signals the class NumberSignal existed. Is there a migration guide for people that used the Signals as they were in the early days?

Also we get the following error:

An asynchronous signal can only share transaction with computed signals and other asynchronous signals that belong to the same tree.

Beforehand this code worked.

We had two Signals that we created with the IN_MEMORY_SHARED SignalFactory. and then we updated the two signals with Signal.runInTransaction. This worked but now it does no longer.

Where should such questions be asked. On some documentation pages you can comment. Like https://vaadin.com/docs/latest/components/avatar. But on the documentation https://vaadin.com/docs/latest/flow/ui-state/shared-signals this does not work, which is kind of strange.

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