[GitHub] [camel-kamelets] oscerd commented on a diff in pull request #1318: Issue-981: Restore optional usage of chatId Configuration Option and correct narrative

["oscerd (via GitHub)" <gi...@apache.org>]

oscerd commented on code in PR #1318:
URL: https://github.com/apache/camel-kamelets/pull/1318#discussion_r1120132350


##########
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:
   If you set a default value, it will be set on the endpoint by default.



-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org