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

Error Handling und Ausgabe ohne msg und/oder format bei allen Nodes möglich #77

Merged
merged 4 commits into from
Jan 15, 2025

Conversation

moellenbeck
Copy link
Member

No description provided.

@moellenbeck moellenbeck merged commit c630de5 into main Jan 15, 2025
2 checks passed
process-engine-ci added a commit that referenced this pull request Jan 15, 2025
# Changelog v1.6.5 (15.01.2025)

Dieser Changelog deckt die Änderungen zwischen folgenden Versionen ab: [v1.6.4 und v1.6.5](v1.6.4...v1.6.5).

Weitere Hinweise befinden sich im Changelog der vorherigen Version: [v1.6.4](https://github.com/5minds/node-red-contrib-processcube/releases/tag/v1.6.4).

## Merged Pull Requests

- #77 Error Handling und Ausgabe ohne `msg` und/oder `format` bei allen Nodes möglich  (merged 15.01.2025)

[skip ci]
@luisthieme
Copy link
Contributor

Der laut Dokumentation korrekte "neue" Weg wäre glaube ich die node.error() Methode nur für logging zu nutzen. Das weitergeben an die Node-RED CatchNode würde dann über die "done()" Funktion funktionieren. Ich weiss nicht ob es einen wirklichen Unterschied zwischen den beiden Wegen gibt, allerdings wird für Versionen >1 der "done()" Weg in der Dokumentation empfohlen. Dokumentation

@moellenbeck
Copy link
Member Author

Der laut Dokumentation korrekte "neue" Weg wäre glaube ich die node.error() Methode nur für logging zu nutzen. Das weitergeben an die Node-RED CatchNode würde dann über die "done()" Funktion funktionieren. Ich weiss nicht ob es einen wirklichen Unterschied zwischen den beiden Wegen gibt, allerdings wird für Versionen >1 der "done()" Weg in der Dokumentation empfohlen. Dokumentation

Danke @luisthieme für den Hinweis, werde ich dann anpassen.

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