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

Better compartmentalization, implement Extension system #813

Open
ix5 opened this issue Mar 15, 2022 · 2 comments
Open

Better compartmentalization, implement Extension system #813

ix5 opened this issue Mar 15, 2022 · 2 comments
Labels
feature needs-contributor Someone needs to implement this. Help wanted! needs-decision Architectural/Behavioral decision by maintainers needed
Milestone

Comments

@ix5
Copy link
Member

ix5 commented Mar 15, 2022

For ease of maintenance, development iteration speed and long-term viability, Isso should untangle its massive inter-dependencies and offer a way for users and developers to implement their own functionality without having to modify the (upstream or forked) core.

Collecting ideas:

Possibly related:

Note: I expect to edit this issue quite a lot

@ix5 ix5 added needs-contributor Someone needs to implement this. Help wanted! feature needs-decision Architectural/Behavioral decision by maintainers needed labels Mar 15, 2022
@ix5 ix5 added this to the 1.0 milestone Mar 15, 2022
@ix5 ix5 pinned this issue Mar 15, 2022
@antont
Copy link

antont commented Mar 21, 2022

BTW on server side, comments.py def _process_fetched_list(self, fetched_list, plain=False): seems like a nice extension point: Isso already does processing to add avatar info based on confs, but that same place could call whatever other processors for the comments list.

I'm adding url string avatar info in our fork somehow now, our users already have the avatar url in their data.

@ix5
Copy link
Member Author

ix5 commented Mar 21, 2022

Yes, that's a great entry point you've identified.

In case you would like to discuss back-and-forth with other contributors taking a look at this, you're also very welcome to join the IRC #isso on Libera.Chat or via Matrix

@BBaoVanC BBaoVanC moved this to Backlog in Isso Todo May 16, 2022
@ix5 ix5 modified the milestones: 1.0, backburner Jun 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature needs-contributor Someone needs to implement this. Help wanted! needs-decision Architectural/Behavioral decision by maintainers needed
Projects
None yet
Development

No branches or pull requests

2 participants