Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(channels): channels package #270

Merged
merged 54 commits into from
Dec 10, 2021
Merged

feat(channels): channels package #270

merged 54 commits into from
Dec 10, 2021

Conversation

samuelmasse
Copy link
Contributor

@samuelmasse samuelmasse commented Dec 2, 2021

Moves all channels to their own package.

Each channel will have an api that will respect these standards :

const telegram = new TelegramChannel()
await telegram.setup(router)

telegram.on('message', async (e) => {
  await telegram.send(e.scope, e.endpoint, { text: 'yoyo' })
})

await telegram.start('gggg', { botToken: 'YOUR_CONFIG' })

Closes DEV-2125

@linear
Copy link

linear bot commented Dec 2, 2021

DEV-2125 Messaging make a channels package

Putting all channels in a single package will allow use to write v2 versions more easily, as well as decoupling the channel code from the server code, and reducing considerably the amount of code in the server package

Base automatically changed from sm-proactive-message to master December 3, 2021 17:48
* delete channel code

* chore(channels): use channels from channels package (#272)

* use telegram from channel package

* use package twilio

* use teams

* use smooch

* example change

* config change

* slack

* messenger

* vonage

* integrate other channels

* telegram +

* fix tests

* fix docker

* initialize

* stop when server closes

* dispatch stop

* webhookRouter

* ChannelTemplate

* prettier

* fix
@samuelmasse samuelmasse marked this pull request as ready for review December 9, 2021 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants