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

HookDispatcher dispatchMultiple doesn't exists anymore #10233

Merged
merged 1 commit into from Sep 3, 2018

Conversation

Projects
None yet
4 participants
@PierreRambaud
Contributor

PierreRambaud commented Aug 31, 2018

Questions Answers
Branch? develop
Description? Cannot change the status of a product in the list product page. This is due to a missing change. dispatchMultple doesn't exists anymore.
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #10231
How to test? Follow issue instructions.

This change is Reviewable

@PierreRambaud PierreRambaud added this to the 1.7.5.0 milestone Aug 31, 2018

@PierreRambaud PierreRambaud changed the title from Fixed #10231 dispatchMultiple doesn't exists anymore to HookDispatcher dispatchMultiple doesn't exists anymore Aug 31, 2018

@marionf marionf self-assigned this Sep 3, 2018

@marionf

This comment has been minimized.

Show comment
Hide comment
@marionf

marionf Sep 3, 2018

Contributor

@PierreRambaud
The progress bar and words seems pixelated with Chrome

capture d ecran_223

With firefox I don't have this problem

Contributor

marionf commented Sep 3, 2018

@PierreRambaud
The progress bar and words seems pixelated with Chrome

capture d ecran_223

With firefox I don't have this problem

@PierreRambaud

This comment has been minimized.

Show comment
Hide comment
@PierreRambaud

PierreRambaud Sep 3, 2018

Contributor

@marionf Not an issue due to this PR, I just fixed hooks :/

Contributor

PierreRambaud commented Sep 3, 2018

@marionf Not an issue due to this PR, I just fixed hooks :/

@marionf marionf added QA ✔️ and removed waiting for QA labels Sep 3, 2018

@marionf marionf removed their assignment Sep 3, 2018

@marionf

This comment has been minimized.

Show comment
Hide comment
@marionf

marionf Sep 3, 2018

Contributor

Fixes #10217

Contributor

marionf commented Sep 3, 2018

Fixes #10217

@marionf

This comment has been minimized.

Show comment
Hide comment
@marionf

marionf Sep 3, 2018

Contributor

Fixes #10224

Contributor

marionf commented Sep 3, 2018

Fixes #10224

@mickaelandrieu

This comment has been minimized.

Show comment
Hide comment
@mickaelandrieu

mickaelandrieu Sep 3, 2018

Contributor

Thanks @PierreRambaud and @marionf for the review!

Contributor

mickaelandrieu commented Sep 3, 2018

Thanks @PierreRambaud and @marionf for the review!

@mickaelandrieu mickaelandrieu merged commit 66c1440 into PrestaShop:develop Sep 3, 2018

1 check passed

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

@PierreRambaud PierreRambaud deleted the PierreRambaud:fix/issue-10231 branch Sep 3, 2018

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