-
Notifications
You must be signed in to change notification settings - Fork 22
Bring Magento template up-to-date #18
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
base: develop
Are you sure you want to change the base?
Conversation
- Update services for Magento 2.4.6, including using MariaDB 10.6 and OpenSearch 2.5 - Remove installation of hirak/prestissimo as we're now using Composer 2 - Add composer auth prompting during bootstrap when auth.json does not exist - Fix TFA creation for recent image updates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will be nice if the script automatic check the .env file and identify if it set elasticsearch or opensearch as search engine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add elasticsearch setup parameters to INSTALL_FLAGS
INSTALL_FLAGS="${INSTALL_FLAGS} \ | ||
--cleanup-database \ | ||
--backend-frontname=backend \ | ||
--backend-frontname=admin \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably pair with an issue in the docs repo to update the references to backend
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also requires update to line 29
URL_ADMIN="https://${TRAEFIK_SUBDOMAIN}.${TRAEFIK_DOMAIN}/backend/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should update this repo's readme.md table for AdminURLs all reference backend as well.
Uh oh!
There was an error while loading. Please reload this page.