-
Notifications
You must be signed in to change notification settings - Fork 109
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
Add Zaso plugins #219
Comments
There is no public repo for these plugins. It'd be good to keep old commits history. |
He has a git account, but is managing everything in a local repository on his PC. |
Is it possible to put it online? |
I'm afraid he can't do that right now. |
OK, then we should remind about it later. |
I have just started to find my way through GitHUB, nevertheless I will take care of this task. |
@MysticJay |
As agreed with @modos189 I'll start with importing "Multi-Project-Extension". That will not have any side effects. |
Are you going to add raw scripts from Zaso homepage?
You can use chat at https://www.reddit.com/r/IITC/ |
I talked with Zaso. He only operates his GIT locally. |
Most of Zaso's tools are explained in this TG-Channel: |
Some thoughts:
This (or similar) functionality should be integrated with IITC to be available directly, without separate plugins. @modos189 |
And I have some general concerns about Zaso (or any 3rd-party) plugins importing to main repo. So simple question: why ever we add any plugin here? Answer seems obvious: plugin is useful. In general I suppose we should not add plugins that have own place in the internet.
I love Zaso plugins. And I'd love to support them if he'll decide to transfer his repo to IITC-CE, and it will be officially. So I strongly dislike the idea to duplicate plugins here 'just in case'. Why? Just add links to them. |
Zaso has ever since seeked to integrate his work into the official IITC. Why it did not make its way I have no idea. The three plugins I am working on right now have the least impact but the most value for all users. You are right, there are hundreds of plugins (I just received a list with over 400). But many address a single missing function or a fancy idea. Some must not be named, are faction specific or even violate TOS. Still they have a "main idea" and that is what we should adopt. The original Bookmarks and Drawtools are Zaso's developments. If his updates to these plugins do not deserve to merge no other new plugin should have. I do not seek to add those as plugin extensions (like they exist today) but to integrate their new functions with the base plugins. Still I have to do that step by step in a matter that does not break anything existing, but is still reviewable for the maintainers. |
As a next step I want to integrate draw-tools-plus with draw-tools. |
This is true. And Zaso's plugins here are extremely useful. If he does not have plans to develop that plugins - then I agree, we can adopt them here.
Oh, this is completely another case. This is best approach.
Sure. Than let's continue related discussion in #221. |
Consider #2, there are many other draw-tools-related PRs waiting.
Do not hurry to integrate MPE support, as it is also not sole existing implementation of multiple projects, and we need to chose the best. |
Reviewing other PRs / Feature Requests will take time. Merging them will be even more complex.
|
@modos189 step2 done. |
Zazo`s is not developing plugins now, but he has given permission to add his plugins to our repository and change if necessary.
He has a lot of interesting plugins. I think we can add most of them to our repository.
The text was updated successfully, but these errors were encountered: