Skip to content

Conversation

@kzu
Copy link
Member

@kzu kzu commented Jun 6, 2025

We should keep the public API to send messages directly through the IWhatsAppClient. This makes the app more reusable in scenarios that aren't implemented directly as functionality pipelines (i.e. async or scheduled jobs that notify users of updates).

The functionality pipeline approach works best for request/reponse scenarios where the user is driving a conversation, but it's not the only scenario we want to cover with this library.

We should keep the public API to send messages directly through the IWhatsAppClient. This makes the app more reusable in scenarios that aren't implemented directly as functionality pipelines (i.e. async or scheduled jobs that notify users of updates).

The functionality pipeline approach works best for request/reponse scenarios where the user is driving a conversation, but it's not the only scenario we want to cover with this library.
@kzu kzu added the enhancement New feature or request label Jun 6, 2025
@kzu kzu requested a review from adalon June 6, 2025 13:11
@kzu kzu enabled auto-merge (rebase) June 6, 2025 13:11
@kzu
Copy link
Member Author

kzu commented Jun 6, 2025

32 passed 32 passed 7 skipped

🧪 Details on Ubuntu 24.04.2 LTS

from dotnet-retest v0.7.1 on .NET 8.0.16 with 💜 by @devlooped

@kzu kzu merged commit 2130ca6 into main Jun 6, 2025
7 checks passed
@kzu kzu deleted the dev/public-extensions branch June 6, 2025 13:13
@devlooped devlooped locked and limited conversation to collaborators Jul 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants