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

Custom prestashop release #96

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Alexandre-Lematre
Copy link
Contributor

@Alexandre-Lematre Alexandre-Lematre commented Apr 18, 2024

Questions Answers
Description? Adds the ability to build a release based on a custom prestashop zip containing a list of modules
Type? new feature
BC breaks? no
Deprecations? no
Sponsor company PrestaShop
How to test? Try to build a release with the PrestaShop Classic Edition zip

build.sh Outdated Show resolved Hide resolved
build.sh Outdated Show resolved Hide resolved
declare PLATFORM; # -- alias for $TARGET_PLATFORM
declare TARGET_IMAGE; # -- docker image name, defaults to "prestashop/prestashop-flashlight"
declare PUSH; # -- set it to "true" if you want to push the resulting image
declare ZIP_SOURCE; # -- the zip to unpack in flashlight
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do actually have an open PR on this one! Can we merge ideas?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure. Can you tell me what PR is this ? I can't find it

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was thinking about that one : #82

@Alexandre-Lematre Alexandre-Lematre force-pushed the feat/classic-edition branch 2 times, most recently from 46f0830 to 9e081f4 Compare May 2, 2024 07:45
declare TARGET_IMAGE; # -- docker image name, defaults to "prestashop/prestashop-flashlight"
declare PUSH; # -- set it to "true" if you want to push the resulting image
declare ZIP_SOURCE; # -- the zip to unpack in flashlight
declare PRE_INSTALLED_MODULES; # -- install modules during zip installation
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this should come along with the build time. It's rather a variable to be declared on top of assets/run.sh

@@ -47,6 +48,8 @@ ARG PHP_VERSION
ARG GIT_SHA
ARG PS_FOLDER=/var/www/html
ARG ZIP_SOURCE
ARG PRE_INSTALLED_MODULES
ENV PRE_INSTALLED_MODULES=$PRE_INSTALLED_MODULES
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it really something you wanna provide at build time?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These modules have to be installed during PrestaShop installation, so, at build time ; the aim being to provide a custom release, shiped with a bunch of modules (like classic and hosted edition)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready for review
Development

Successfully merging this pull request may close these issues.

None yet

2 participants