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

Delay node documentation - 'flush' property is missing some details in the german translation #4748

Closed
aikitori opened this issue Jun 4, 2024 · 1 comment · Fixed by #4762

Comments

@aikitori
Copy link

aikitori commented Jun 4, 2024

As discussed here:
https://discourse.nodered.org/t/on-demand-delay-node/88466/10?u=kitori

In the german translation there is missing some details what you can do with the msg property flush

English:

If the received message has this property set to a numeric value then that many messages will be released immediately. If set to any
other type (e.g. boolean), then all outstanding messages held by the node are sent immediately.

German:
Wenn bei der empfangenen Nachricht diese Eigenschaft auf einen beliebigen Wert gesetzt ist, werden alle im Node gepufferten Nachrichten sofort gesendet.

The highlighted part is missing in the German translation

grafik
grafik


This sentence describe the same like the english one:

German:
Wenn bei der empfangenen Nachricht diese Eigenschaft auf einen numerischen Wert gesetzt ist, wird diese Anzahl an Nachrichten sofort gesendet. Wenn ein anderer Typ gesetzt ist (z.B. Boolean), werden alle in der Node gepufferten Nachrichten gesendet.

Edit:

And rate is missing:

rate
Setzt die Verzögerung in Millisekunden zwischen den Nachrichten. Diese Node überschreibt die bestehende Verzögerung die in der Node konfiguration, wenn die empfangende Nachricht msg.rate in Millisekunden enthält. Dies trifft nur zu, wenn in der Node konfiguriert ist, das empfangene Nachrichten den konfigurierten Wert überschreiben können.

@GogoVega
Copy link
Contributor

GogoVega commented Jun 5, 2024

Hi,
Could you make a PR to include this changes? The file is here.
Thank you.

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 a pull request may close this issue.

2 participants