feat: Add errors, messages, and results properties
#506
Conversation
daogrady
changed the title
errors, messages, and results propertieserrors, messages, and results properties
apis/events.d.ts
Outdated
|
|
||
| messages: {message: string, numericSeverity: levels}[] | ||
|
|
||
| errors: {message: string, stack: string}[] |
There was a problem hiding this comment.
Not sure what should be visible in types. There are further properties like numericSeverity, code, target.
There was a problem hiding this comment.
I got these values from experimenting with a bookshop project.
I'll gladly add any properties that should be available (this also showcases why me experimentally determining them is vastly inferior to a runtime expert providing them with confidence 😕)
There was a problem hiding this comment.
It depends on what you provide into req.error() as parameters. Try out req.error(code, message, target). For example req.error (400, 'Invalid input', 'some_field').
There was a problem hiding this comment.
There was a problem hiding this comment.
I see. I have added the fields accordingly, thanks for pointing that out!
Fixes #423
https://cap.cloud.sap/docs/node.js/events#req-notify
https://cap.cloud.sap/docs/node.js/events#req-reply-results