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

Instantiate AdminKernel #679

Merged
merged 16 commits into from
Mar 26, 2024
Merged

Conversation

ga-devfront
Copy link
Contributor

Questions Answers
Description? Instantiate the appropriate kernel based on the environment
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? Fixe issue #35029
Sponsor company @PrestaShopCorp
How to test?

Hlavtox

This comment was marked as outdated.

@ga-devfront

This comment was marked as outdated.

@M0rgan01 M0rgan01 removed the Waiting for review Waiting for review label Mar 22, 2024
@AureRita AureRita self-assigned this Mar 26, 2024
@AureRita
Copy link
Contributor

Hi @ga-devfront

Thank you for your PR, Currently, I tried to test your PR. The installation going well until I want to go in my admin panel

This is the error showed :
image

Tested on 8.1.5 and 8.0.5

Do you know how we can improve this ?

Waiting for feedback

Copy link
Contributor

@AureRita AureRita left a comment

Choose a reason for hiding this comment

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

Seen with @MatShir,

Because this PR correct the initial issue and let us discover another one, we'll validate this PR and create a new issue, So this PR is QA ✔️

Thank you @ga-devfront for your correction !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

8 participants