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

support submodule and zip installation #4

Open
bnomei opened this issue May 13, 2019 · 6 comments
Open

support submodule and zip installation #4

bnomei opened this issue May 13, 2019 · 6 comments
Labels
help wanted Feel free to make a pull request implementing this featured

Comments

@bnomei
Copy link

bnomei commented May 13, 2019

to be listed at the offical kirby 3 plugin list you need to allow all three installation modes (composer, gitmodule and zip).

please check out the official example repo for a plugin with composer dependencies

it boils down to a special gitignore file and a reduced commited vendor folder. i know it's not the clean php composer way but it seems to be the best choice for kirby plugins.

apart from that i will certainly try out your plugin asap. ❤️

@thathoff
Copy link
Owner

Thanks! The question is: How can we support all providers by not requiring the installation via composer. I currently don’t have a proper solution for this.

@bnomei
Copy link
Author

bnomei commented May 13, 2019

https://getkirby.com/docs/guide/plugins/plugin-setup-composer

you already have the required libs to composer.json. you just need to update the gitignore file to the provided example from the docs. this will allow you to commit a reduced version of the vendor folder. that is all you have to do.

@thathoff
Copy link
Owner

Yeah I know. But the problem is the plugin itself is basically useless without installing additionally dependencies. To use Google as an identity provider you have to manually add the league/oauth2-google package or for Facebook the league/oauth2-facebook. So I think the only option would be to include the most common providers in the distributed vendor folder and only support additional providers via composer (which I actually don’t like very much) ;).

@bnomei
Copy link
Author

bnomei commented May 13, 2019

ah understood. since creating custom kirby plugins is more or less easy but at least well documented you could provide an example how to create a simple one (with composer) for the auth lib instead of just saying one needs to use composer for everything.

add to readme something like this:

  • download zip of you plugin, submodule or composer (with altered gitignore)
  • if composer just require auth lib or else
  • create a custom plugin in site/plugins using composer setup and require lib there.

@thathoff
Copy link
Owner

thathoff commented May 13, 2019

Hmm, I like the idea of a custom plugin. But nevertheless you always need composer so I don’t see a real advantage in providing other installation methods. But we will definitely think about a nice solution to this issue. Thank’s for your input!

@bnomei
Copy link
Author

bnomei commented May 18, 2019

it happend to me that people try to use the zip and then file an issue. if you adhere to the pluginkit way of commiting the vendor folder your plugin is at least working standalone. someone else could then wrap a simple plugin for custom auth and plush that. others could just use the zips of these to get started. they would not need composer then.

@thathoff thathoff added the help wanted Feel free to make a pull request implementing this featured label Jun 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Feel free to make a pull request implementing this featured
Projects
None yet
Development

No branches or pull requests

2 participants