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

Button erase all doesn't work for logs & email pages #14686

Closed
Stephane1874 opened this issue Jul 16, 2019 · 3 comments

Comments

@Stephane1874
Copy link

commented Jul 16, 2019

Describe the bug

Hello the team,

Go to the advanced settings page> logs, tried to delete the logs, the button does not work.

To Reproduce
Steps to reproduce the behavior:

  1. Go to advanced settings > logs
  2. Click on Delete all
  3. See error

Screenshots
supprimer

Additional information
PrestaShop version: 1.7.6
PHP version: 7.1.3

@khouloudbelguith

This comment has been minimized.

Copy link
Contributor

commented Jul 16, 2019

Hi @Stephane1874,

Thanks for your report.
I manage to reproduce the issue with PS1.7.6.0 fresh install & after upgrade from PS1.7.5.
https://drive.google.com/file/d/1kRxIzVAU3WIKNYQ5dxY-wz-sCtMn64yF/view
In the PS1.7.5.1 => OK
I’ll add this to the debug roadmap so that it’s fixed. If you have already fixed it on your end or if you think you can do it, please do send us a pull request!
Thanks!

@marionf marionf added this to Backlog in PrestaShop 1.7.6 via automation Jul 16, 2019

@marionf marionf moved this from Backlog to To do in PrestaShop 1.7.6 Jul 16, 2019

@marionf marionf changed the title Can't delete logs Button erase all doesn't work for logs & email pages Jul 17, 2019

@marionf marionf added the Email label Jul 17, 2019

@marionf

This comment has been minimized.

Copy link
Contributor

commented Jul 17, 2019

Same issue on email page, erase all button doesn't work
But if you select all and delete with bulk action it's working

@matks matks self-assigned this Jul 17, 2019

@matks matks moved this from To do to To be tested in PrestaShop 1.7.6 Jul 17, 2019

@matks matks moved this from To be tested to To be reviewed in PrestaShop 1.7.6 Jul 17, 2019

@matks matks moved this from To be reviewed to To be tested in PrestaShop 1.7.6 Jul 17, 2019

@sarahdib sarahdib added Fixed and removed To Do labels Jul 17, 2019

@sarahdib sarahdib added this to the 1.7.6.1 milestone Jul 17, 2019

@sarahdib sarahdib moved this from To be tested to To be merged in PrestaShop 1.7.6 Jul 17, 2019

@matks

This comment has been minimized.

Copy link
Contributor

commented Jul 18, 2019

Fixed by #14701

@matks matks closed this Jul 18, 2019

PrestaShop 1.7.6 automation moved this from To be merged to Done Jul 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.