-
Notifications
You must be signed in to change notification settings - Fork 15
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
Use Magento 2.4 #24
base: master
Are you sure you want to change the base?
Use Magento 2.4 #24
Conversation
Just reviewed this MR and hit some strange merge conflicts when testing merging into main. If |
Interestingly, I checked the PR environment, and it's throwing an error.
Checked the deploy log and it shows it failed to install:
<snip>
Tried redeploying, same error. |
This is likely due to the PR environment starting out as 2.3 rather than a blank project. Special steps would need to be taken to upgrade from 2.3 to 2.4. To test this PR you'd want to start with a blank environment. |
I'm looking to upgrade to 2.4, is this ticket still active? |
It's really important to get this done. Current is Magento 2.3.7, which went out of support a year ago
|
Description (*)
Upgrade template to Magento 2.4 with Elasticsearch 7.7 and Composer 2
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
On Lando Development
This is 99% Lando compatible. The only issue is that after
lando start
you mustlando restart
before the Elasticsearch is exposed to the app. This is a Lando issue, not a Magento or template issue.This also means that if you are using Lando for local development, you'll need to manually
lando ssh -c "php deploy.php"
for the Magento setup to run locally.PR Commits
Lots of files are changed in this PR. Sorry, they're not squashed. Here's what to expect:
setup:install
bug.Contribution checklist (*)