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

Opening the tdesktop architecture: planning: pure virtual class chat #5620

Closed
nonlin-lin-chaos-order-etc-etal opened this issue Jan 26, 2019 · 3 comments

Comments

@nonlin-lin-chaos-order-etc-etal
Copy link

nonlin-lin-chaos-order-etc-etal commented Jan 26, 2019

Folks, what do you think about this: libregram#1 : I'd like to create a pull request into this (your) tdesktop repo, with changes/additions described below (no code changes currently done, I plan this). I'd like to add allowing for custom implementations for every chat (for every tdesktop gui chatlist entry): create pure virtual class chat with horde of pure virtual methods. And implement standard tg chats through this abstract class.

Will you merge this work (pull request) in, if perfectly done? I've not yet studied the tdesktop code.

@nonlin-lin-chaos-order-etc-etal nonlin-lin-chaos-order-etc-etal changed the title Opening the tdesktop architecture: plugins for chats Opening the tdesktop architecture: planning: plugins for chats Jan 26, 2019
@nonlin-lin-chaos-order-etc-etal nonlin-lin-chaos-order-etc-etal changed the title Opening the tdesktop architecture: planning: plugins for chats Opening the tdesktop architecture: planning: pure virtual class chat Jan 28, 2019
@john-preston
Copy link
Member

I'm not sure this is possible in tdesktop. I think you'd better look into the code first for better understanding what you want to do there exactly. For example, there is no such thing as "gui chatlist entry" and I can't imagine what you could really customize in chats list through plugins.

@stek29
Copy link
Contributor

stek29 commented Jan 28, 2019

And I don't think such virtual class based approach is a good way to implement plugin system.
Even if it was, this is a perfect example of PRs which don't really fit well with "What contributions are accepted?"

Anyway, see #472.

@github-actions
Copy link

github-actions bot commented Mar 6, 2021

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants