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

Refactor Channel to more clearly separate "single session per user" from "multiple session per user" #462

Open
snopoke opened this issue Jun 14, 2024 · 0 comments

Comments

@snopoke
Copy link
Collaborator

snopoke commented Jun 14, 2024

Slack and Web channels are 'multi-session' per user where each user / participant can have multiple active sessions. Other channels like Telegram, Whatsapp etc can only have a single active session.

Consider splitting the ChannelBase class to more clearly delineate the differences.

Also refactor test_base_channel_behaviour. Currently we use the telegram channel as a vehicle to test this, but ideally we should be using all channel types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

1 participant