wp-media / imagify-plugin

Speed up your website with lighter images without losing quality.
https://imagify.io
71 stars 25 forks source link

Improve information related to out of quota on the bulk #714

Closed jbma closed 1 year ago

jbma commented 1 year ago

Is your feature request related to a problem? Please describe. Once the bulk optimization is launched and all the quota is consumed, we don't directly display the information about the quota. Instead it looks like the bulk is continuing.

That's for example how this is displayed on the bulk (the user has consumed all his quota) : image

Describe the solution you'd like Instead we should directly use the popin we were using before to warn the user about the quota and provide him a solution.

CrochetFeve0251 commented 1 year ago

@jbma to confirm we need to show the exact same popup he got at 20% storage remaining again even if the customer already clicked on it by the past to dismiss it? Or we need a new one looking the same way.

jbma commented 1 year ago

In the previous version when the quota was 0, when we tried to launch the bulk optimization or when running (we had to stay on the same page, wasn't async) a popin was displayed when launching the bulk or when it was running.

I guess here the best solution would be to display the popin each time we go the bulk when the quota is 0

CrochetFeve0251 commented 1 year ago

Scope a solution

To activate the popup when the quota is over we need to modify two classes to add the following logic:

PS: @Tabrisrp while doing that I got no popup displaying due to no results found in renew_almost_over_quota_notice method, could you confirm that value would be filled if we reach 80% usage?