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

Fix ajax refresh for listing pages #6674

Merged
merged 3 commits into from Oct 17, 2016

Conversation

Projects
None yet
6 participants
@julienbourdeau
Contributor

julienbourdeau commented Oct 13, 2016

Questions Answers
Branch? develop
Description? Some data weren't updated properly + markup was in core.js instead of theme.js
Type? bug fix
Category? FO
BC breaks? no
Deprecations? no
Fixed ticket? http://forge.prestashop.com/browse/BOOM-1644
How to test? Use the faceted search in the sidebar and see if it works
@aleeks

This comment has been minimized.

Show comment
Hide comment
@aleeks

aleeks Oct 14, 2016

Contributor

@julienbourdeau can you rebase please, i'm sorry!

Contributor

aleeks commented Oct 14, 2016

@julienbourdeau can you rebase please, i'm sorry!

@maximebiloe

This comment has been minimized.

Show comment
Hide comment
@maximebiloe

maximebiloe Oct 17, 2016

Contributor

Don't forget to report it on StarterTheme please

Contributor

maximebiloe commented Oct 17, 2016

Don't forget to report it on StarterTheme please

Show outdated Hide outdated themes/classic/templates/catalog/_partials/products.tpl
@aleeks

aleeks approved these changes Oct 17, 2016

@aleeks aleeks self-assigned this Oct 17, 2016

@aleeks

This comment has been minimized.

Show comment
Hide comment
@aleeks

aleeks Oct 17, 2016

Contributor

thank you @julienbourdeau !

Contributor

aleeks commented Oct 17, 2016

thank you @julienbourdeau !

@aleeks aleeks merged commit 140bdce into PrestaShop:develop Oct 17, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@julienbourdeau julienbourdeau deleted the julienbourdeau:feat/improve-ajax-for-listing branch Oct 18, 2016

@aleeks aleeks removed their assignment Oct 19, 2016

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