-
Notifications
You must be signed in to change notification settings - Fork 80
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
Format availability for the new style #658
Conversation
@florine2623 This can be tested standalone, the |
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.
Hello @Hlavtox ,
I have a rebase going on when I install this PR.
Am I missing something ?
Waiting for your feedback ^^
Thanks!
@florine2623 No idea, but the core doesn't have anything to do with this PR. :-) I suggest using a built version of hummingbird and just changing the one file manually. |
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.
Hello @Hlavtox ,
This has been tested with PrestaShop/PrestaShop#37018.
As I couldn't install the PR properly, I followed your suggestion to change just the file changed on my template.
More details on the test of this PR : PrestaShop/PrestaShop#37018 (review)
Thanks !
It is QA ✅
PR merged, well done! Message to @PrestaShop/committers: do not forget to milestone it before the merge. |
Thank you @florine2623! :-) |
Related core PRs
How it looks
In stock
Available on backorder
Not available (+ a new availability submessage)