send message type image #1690
send message type image #1690
Conversation
| @@ -302,6 +313,27 @@ class WhatsAppCloudApiService(private val apiClient: WhatsAppCloudApiClient) { | |||
| } | |||
| } | |||
|
|
|||
| private fun replaceWithRealMassageImageId( | |||
There was a problem hiding this comment.
| private fun replaceWithRealMassageImageId( | |
| private fun replaceWithRealMessageImageId( |
| @@ -328,7 +340,7 @@ fun <T : Bus<T>> T.whatsAppCloudTextParameterTemplate( | |||
| typeParameter: CharSequence?, | |||
| textButton: CharSequence? | |||
| ): TextParameter = TextParameter( | |||
| type = ParameterType.valueOf(translate(typeParameter).toString()), | |||
| type = ParameterType.valueOf((typeParameter).toString()), | |||
There was a problem hiding this comment.
I believe this change is a duplicate of #1683
|
|
||
| data class WhatsAppCloudSendBotImageMessage( | ||
| data class WhatsAppCloudSendBotImageMessage ( |
| val image: WhatsAppCloudBotImage, | ||
| override val recipientType: WhatsAppCloudBotRecipientType, | ||
| override val to: String, | ||
| ) : WhatsAppCloudSendBotMessage(WhatsAppCoudBotMessageType.image) |
There was a problem hiding this comment.
Semi-related to this PR: this class name has a typo
There was a problem hiding this comment.
je ne pense pas
There was a problem hiding this comment.
WhatsAppCoudBotMessageType instead of WhatsAppCloudBotMessageType?
|
|
||
| override fun toGenericMessage(): GenericMessage = | ||
| GenericMessage( | ||
| texts = mapOf(GenericMessage.TEXT_PARAM to "image"), |
There was a problem hiding this comment.
The GenericMessage class has a dedicated attachments field for this purpose, you can refer to WebModels#WebImage.toGenericMessage for an example:
tock/bot/connector-web/src/main/kotlin/WebModels.kt
Lines 58 to 65 in 6cd5158
There was a problem hiding this comment.
Note: this can be done in a future PR
| fun BotBus.whatsAppCloudImage( | ||
| id: String, | ||
| link: String? = null, | ||
| caption: String? = null, |
There was a problem hiding this comment.
Shouldn't this be a CharSequence, so that we can translate it?
SNCFamiraBenkaci
force-pushed
the
1689-add-send-message-image
branch
2 times, most recently
from
d530156
to
667c11f
Compare
SNCFamiraBenkaci
force-pushed
the
1689-add-send-message-image
branch
from
667c11f
to
c3e8d5f
Compare
SNCFamiraBenkaci
force-pushed
the
1689-add-send-message-image
branch
from
c3e8d5f
to
61d4a7c
Compare
* send message type image * send message type image --------- Co-authored-by: BENKACI Amira <[email protected]>
#1592