manstis commented on code in PR #1318:
URL: https://github.com/apache/camel-kamelets/pull/1318#discussion_r1120216302
##########
kamelets/telegram-sink.kamelet.yaml:
##########
@@ -60,8 +60,14 @@ spec:
- urn:camel:group:credentials
chatId:
title: Chat ID
- description: The Chat ID to where you want to send messages by
default. Optional if the `chat-id` / `ce-chatid` header is used.
+ description: |-
+ The Chat ID to where you want to send messages by default.
+
+ The Chat ID must be provided for every message; either as a
Configuration Option or a `chat-id` / `ce-chatid` header.
+
+ If the Chat ID is not explicitly provided the default value is used
which may lead to delivery failures.
type: string
+ default: dummy
Review Comment:
OK. I've restored the `required` nature of the `chatId` parameter and
hopefully made the documentation bullet proof!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
