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

Remove logo #87

Merged
merged 2 commits into from Feb 16, 2024
Merged

Remove logo #87

merged 2 commits into from Feb 16, 2024

Conversation

Hlavtox
Copy link
Contributor

@Hlavtox Hlavtox commented Feb 16, 2024

Questions Answers
Description? Revert #79, default themes are not ready for it. We can add it back with a configuration option.
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? Fixes a blocked release
How to test? No need to test

@Hlavtox Hlavtox added this to the 2.2.0 milestone Feb 16, 2024
@Hlavtox
Copy link
Contributor Author

Hlavtox commented Feb 16, 2024

Ping @kpodemski @MatShir

@MatShir
Copy link

MatShir commented Feb 16, 2024

Let's unblock the release ! And invest more time in the theme before such feature.

@ps-jarvis ps-jarvis added the waiting for QA Status: Waiting for QA feedback label Feb 16, 2024
@kpodemski kpodemski removed the waiting for QA Status: Waiting for QA feedback label Feb 16, 2024
@jolelievre
Copy link
Contributor

Merged without QA because it is a revert from #79

@jolelievre jolelievre merged commit c5080bf into PrestaShop:dev Feb 16, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
5 participants