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] Merge Push, Email & SMS modules #1172

Open
kkopanidis opened this issue Sep 25, 2024 Discussed in #122 · 0 comments · May be fixed by #1160
Open

[FEAT] Merge Push, Email & SMS modules #1172

kkopanidis opened this issue Sep 25, 2024 Discussed in #122 · 0 comments · May be fixed by #1160
Labels
discussion enhancement New feature or request
Milestone

Comments

@kkopanidis
Copy link
Contributor

Discussed in #122

Originally posted by kkopanidis February 8, 2022
Is your feature request related to a problem? Please describe.
Currently conduit spins up multiple containers that don't really make sense to be separate, since they can't practically scale independently. Email, push and sms utilise different external providers for their operations, but they fulfil a common goal of outgoing operations with templates etc.

Describe the solution you'd like
I think it'd be best for these to be merged into a singular "communications" module, that handles all outgoing user communications with either channel. This will allow for even cool things like "reach the user by any mean necessary", and incorporate more channels ex. viber, whatsapp etc.

Additional context
This particular module merge doesn't have any scalability arguments i think, since basically their capacity dependes 80-90% to the external services and merging them shouldn't provide any significant benefits or drawbacks on that part. But, it'd definitely save up some ram and cpu in the deployments.

@kkopanidis kkopanidis added enhancement New feature or request discussion labels Sep 25, 2024
@kkopanidis kkopanidis added this to v0.17 Sep 25, 2024
@kkopanidis kkopanidis moved this to Todo in v0.17 Sep 25, 2024
@kkopanidis kkopanidis moved this from Todo to In Progress in v0.17 Sep 25, 2024
@kkopanidis kkopanidis added this to the v0.17 milestone Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion enhancement New feature or request
Projects
Status: In Progress
1 participant