Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(NbDialogService): support passing values to input/model signals #3278

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

junglerobba
Copy link

This makes an exception for input and model signals in the DialogConfig context, so the typehint will show as the type of the signal's value, but internally it will create a signal with the passed value. Technically these will just be readonly signals instead of input or model signals, but since components used as a dialog do not support regular outputs anyway, it should not matter. When used in a regular context the input signal will not be overwritten and work as usual.

Please read and mark the following check list before creating a pull request:

Short description of what this resolves:

closes #3256 (I would not classify this as a bug but a missing feature instead)

This makes an exception for input and model signals in the DialogConfig
context, so the typehint will show as the type of the signal's value,
but internally it will create a signal with the passed value.
Technically these will just be readonly signals instead of input or model
signals, but since components used as a dialog do not support regular
outputs anyway, it should not matter. When used in a regular context the
input signal will not be overwritten and work as usual.

closes akveo#3256
(I would not classify this as a bug but a missing feature instead)
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.

NbDialogService has problem with inputSignal and ModelSignal
1 participant