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 installing of addons #83

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dploeger
Copy link

This adds a feature to the entrypoint script that installs Redaxo addons either from redaxo.org or URLs to zip files.

@dploeger
Copy link
Author

dploeger commented Mar 5, 2024

Tagging @schuer as you were busy in the commits. 😄 Would this PR make sense?

@schuer
Copy link
Member

schuer commented Mar 9, 2024

Hey @dploeger, danke für den PR und den Ping. Ich habe eine Weile drüber nachdenken müssen, wie ich das Feature finde. Am Ende bin ich persönlich eher Contra, will der Sache aber auch nicht im Weg stehen — vor allem, weil ich eigentlich nichts mehr mit REDAXO zu tun habe.

Zum Feature: Es könnte sehr nützlich sein. Aber eigentlich gibt es bereits das custom-setup.sh-Skript, das für den Zweck vorgesehen ist, am Ende der REDAXO-Installation weitere Schritte auszuführen, um beispielsweise AddOns zu installieren. Die Demos machen Gebrauch davon, etwa so: https://github.com/FriendsOfREDAXO/docker-demos/blob/main/images/base/custom-setup.sh

Allerdings fällt mir im Zuge dieses PRs auf, dass wir dieses Feature mit dem custom-setup.sh besser mal in der README erklären sollten. Es wird zwar im Tutorial REDAXO mit Docker beschrieben, aber es fehlt hier in diesem Repo. 😶

Jedenfalls, ich will deinem PR nicht im Weg stehen, aber ich halte das Skript für sinnvoller. Vor allem auch im Hinblick darauf, dass REDAXO 6 nicht mehr den Installer nutzen wird, sondern AddOns mittels Composer installiert werden.

Wir können gerne weitere Meinungen aus der Community einholen.

@dploeger
Copy link
Author

Hallo @schuer !

Danke für die umfangreichen Informationen. Meine Idee war, dass die Installation von Plugins im Standard sein sollte und nicht über einen Custom-Container mit custom-setup.sh reinlaufen sollte.

Hintergrund war, dass ich unsere Website containerisieren wollte und auch für die Entwicklungsumgebung schon ein komplettes Featureset konfigurieren wollte ohne dafür extra ein eigenes Container-Image irgendwo hin zu deployen.

Das Feature in diesem PR kann man auch für Redaxo 6 recht leicht in eine Composer-Basis gießen (ist ja im Grunde genommen dann nur ein Aufruf von composer require)

Deine Idee, weitere Meinungen einzuholen finde ich klasse. Ich bin jetzt nur der Admin unserer Site und nicht wirklich in der Redaxo-Community beheimatet. Hast Du die Möglichkeit, hierfür Meinungen einzuholen?

Danke!

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