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

Install plugins automatically #64

Closed
wants to merge 2 commits into from

Conversation

tyzbit
Copy link
Contributor

@tyzbit tyzbit commented Sep 17, 2023

  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

Introduce GRAV_PLUGINS, which can be set to a comma-separated list of plugins to install automatically.

Benefits of this PR and context:

Configuring sites is one of the most tedious aspects of creating sites, this PR would allow you to specify plugins to install automatically as an environment variable so you don't have to run CLI commands or use the Admin UI.

How Has This Been Tested?

Ran on a live site with GRAV_PLUGINS set to a plugin that was already installed and one that was not installed. The script handled the already installed plugin correctly and also installed the new plugin with no issue.

I built an admin variant of this with base version 1.7.42.3 as an image on Docker Hub for testing:

tyzbit/grav:install-plugins-automatically

Source / References:

N/A

DIscussion

Would this be useful to install the admin plugin and allow us to publish just one version instead of an additional admin version?

@tyzbit tyzbit requested a review from dsavell as a code owner September 17, 2023 22:13
@dsavell
Copy link
Owner

dsavell commented Oct 3, 2023

Heya,

Thank you for your contribution, I have finally been able to get round to getting this image updated.

Please see #65 for these changes I have consolidated.

@dsavell dsavell closed this Oct 3, 2023
@tyzbit tyzbit deleted the install-plugins-automatically branch October 3, 2023 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants