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

Add Zaso plugins #219

Open
2 of 3 tasks
modos189 opened this issue Jul 9, 2019 · 19 comments
Open
2 of 3 tasks

Add Zaso plugins #219

modos189 opened this issue Jul 9, 2019 · 19 comments
Labels
development general development issue

Comments

@modos189
Copy link
Contributor

modos189 commented Jul 9, 2019

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.

@johnd0e
Copy link
Contributor

johnd0e commented Jul 9, 2019

There is no public repo for these plugins.
May be he has private one?

It'd be good to keep old commits history.
Also I know that not all his plugins are published.

@modos189
Copy link
Contributor Author

modos189 commented Jul 9, 2019

He has a git account, but is managing everything in a local repository on his PC.
He thinks he will be back later this year, but currently has too much RL.

@johnd0e
Copy link
Contributor

johnd0e commented Jul 9, 2019

managing everything in a local repository on his PC.

Is it possible to put it online?
If there is some unrelated stuff then we can filter it out.

@modos189
Copy link
Contributor Author

modos189 commented Jul 9, 2019

I'm afraid he can't do that right now.

@johnd0e
Copy link
Contributor

johnd0e commented Jul 9, 2019

OK, then we should remind about it later.

@MysticJay
Copy link
Contributor

I have just started to find my way through GitHUB, nevertheless I will take care of this task.
Give me a couple of days to get this beast under control.

@johnd0e
Copy link
Contributor

johnd0e commented Jul 10, 2019

@MysticJay
I like your enthusiasm!
What exactly are you going to do for this issue?

@MysticJay
Copy link
Contributor

MysticJay commented Jul 10, 2019

As agreed with @modos189 I'll start with importing "Multi-Project-Extension". That will not have any side effects.
Can I get you on TG?

@johnd0e
Copy link
Contributor

johnd0e commented Jul 10, 2019

I'll start with importing "Multi-Project-Extension"

Are you going to add raw scripts from Zaso homepage?
Or you have original git repo with full commits history?

Can I get you on TG?

You can use chat at https://www.reddit.com/r/IITC/

@MysticJay
Copy link
Contributor

I talked with Zaso. He only operates his GIT locally.
So yes, I will add the scripts one by one.
Without history. In case it will be available one day we can still add that.

@MysticJay
Copy link
Contributor

Most of Zaso's tools are explained in this TG-Channel:
https://t.me/ZasoItems

@johnd0e
Copy link
Contributor

johnd0e commented Jul 11, 2019

Some thoughts:

This (or similar) functionality should be integrated with IITC to be available directly, without separate plugins.
I am no sure thought that in exact implementation. This needs to be investigated.

@modos189
So do you mean that we should add only these 3 plugins?

@johnd0e
Copy link
Contributor

johnd0e commented Jul 11, 2019

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.
But: should we really add here every useful plugin we can found in the net? There are dozens or hundreds of wonderful plugins.

In general I suppose we should not add plugins that have own place in the internet.

  1. It is confusing: people see same plugins in different places and wondering what is the difference.
    There should be single place where plugin developed. If author actually transfer his abandoned plugin to us - he could state that explicitly, at former homepage.
    And there will be even worse if author continues developing on his homepage (so our version becomes outdated, or forks develop in different ways).

  2. Adding plugin to main repo we have to bring in some value.
    And what is the value if plugin was always available from it's own homepage (which is better than ours, has screenshots, etc)?
    We can add value by actually developing that plugin, enhancing, fixing bugs. If is is not in case - I do not see point of such additions.
    Yes, plugin is great. It's worth 'must-have' section of IITC site.

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.
Or we can add plugins, that we actually enhancing.

So I strongly dislike the idea to duplicate plugins here 'just in case'. Why? Just add links to them.

@MysticJay
Copy link
Contributor

MysticJay commented Jul 11, 2019

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.
Switching between different DRAWS and sets of Bookmarks is most essential for anyone planning OPs, esp. when different OPs are planned at the same time.

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.
M.

@MysticJay
Copy link
Contributor

As a next step I want to integrate draw-tools-plus with draw-tools.
Adding MPE-Support and the new functions (like merge) will make the review difficult.
So should I start with MPE-Support
or should I start with the new functions?

@johnd0e
Copy link
Contributor

johnd0e commented Jul 11, 2019

Switching between different DRAWS and sets of Bookmarks is most essential for anyone planning OPs

This is true. And Zaso's plugins here are extremely useful.
But why duplicate them in our repo? They have own homepage.
We might link to it.

If he does not have plans to develop that plugins - then I agree, we can adopt them here.
In this case it'll make sense to remove download links from old homepage (like is done with Bookmarks).

I do not seek to add those as plugin extensions (like they exist today) but to integrate their new functions with the base plugins

Oh, this is completely another case. This is best approach.

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.

Sure. Than let's continue related discussion in #221.

@johnd0e
Copy link
Contributor

johnd0e commented Jul 11, 2019

As a next step I want to integrate draw-tools-plus with draw-tools.

Consider #2, there are many other draw-tools-related PRs waiting.
It is a big work - to analyse all, and decide which merge order would be optimal.

So should I start with MPE-Support
or should I start with the new functions?

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.
I suppose then better start with Plus-functions (also considering #2-alternatives).

@MysticJay
Copy link
Contributor

MysticJay commented Jul 11, 2019

Reviewing other PRs / Feature Requests will take time. Merging them will be even more complex.
Thus I'd prefer the following:

  • 1) create an alternative to draw-tools ("Draw2-beta") that will allow to switch, while it would not break the pending PRs.
  • 2) enable Draw2 for MPE (to allow showing the fictionality)
  • 3) review draw-related PRs and create a list of ideas from them
  • 4) decide and merge

@MysticJay MysticJay mentioned this issue Jul 11, 2019
@johnd0e johnd0e added the development general development issue label Jul 12, 2019
@MysticJay
Copy link
Contributor

@modos189 step2 done.

@johnd0e johnd0e changed the title Add Zazo plugins Add Zaso plugins Aug 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development general development issue
Projects
None yet
Development

No branches or pull requests

3 participants