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.

Sign up for GitHub

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

Jump to bottom

Fix the show/hide notification option #6489

Merged
merged 2 commits into from Sep 28, 2016
Merged

Fix the show/hide notification option #6489

merged 2 commits into from Sep 28, 2016

Conversation

antoin-m
Copy link
Contributor

Questions Answers
Branch? develop
Description? It should be possible to hide/show some/all notification types
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? http://forge.prestashop.com/browse/BOOM-1568
How to test?
  • In Advanced Parameters / Administration disable some notifications
  • Go to another page
  • The notification type you disabled shouln't appear
  • Disable all notifications
  • Go to another page
  • The whole notification center shouln't appear

@antoin-m antoin-m added WIP and removed WIP labels Sep 26, 2016
@vTerenti vTerenti added the QA ✔️ Status: check done, code approved label Sep 27, 2016
@thierrymarianne thierrymarianne added the Waiting for rebase Status: action required, waiting for rebase label Sep 28, 2016
@antoin-m antoin-m removed the Waiting for rebase Status: action required, waiting for rebase label Sep 28, 2016
@thierrymarianne thierrymarianne merged commit 1cf02c4 into PrestaShop:develop Sep 28, 2016
@thierrymarianne
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
QA ✔️ Status: check done, code approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@antoin-m @thierrymarianne @vTerenti