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

Put back double quotes to fix inclusion error #652

Merged
merged 2 commits into from
Nov 17, 2024

Conversation

maxime-aknin
Copy link
Contributor

Questions Answers
Description? Fix Bug in smarty template inclusion
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? Fixes this ticket
Sponsor company
How to test? Go visit a category page in dev mode

@ps-jarvis
Copy link

Hello @maxime-aknin!

This is your first pull request on hummingbird repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

@florine2623 florine2623 added the Blocked Status: The issue is blocked by another task label Oct 1, 2024
@florine2623
Copy link

Waiting for the author's feedback on the related issue

@Hlavtox Hlavtox removed the Blocked Status: The issue is blocked by another task label Nov 6, 2024
@Hlavtox
Copy link
Contributor

Hlavtox commented Nov 6, 2024

@florine2623 Can we merge this?

@kpodemski kpodemski merged commit 68fee1e into PrestaShop:develop Nov 17, 2024
6 checks passed
@ps-jarvis
Copy link

PR merged, well done!

Message to @PrestaShop/committers: do not forget to milestone it before the merge.

@kpodemski
Copy link
Contributor

I tested it on my local installation, where I encountered this error. It works as expected.

Thank you @maxime-aknin

@kpodemski kpodemski added this to the v0.2.1 milestone Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
7 participants