Bin Management Improvements - Deleted pages, attachments

[vmassol]

@amilica

There’s no input search for Title (maybe this is just a version bug - 16.10.8)

No it’s not. I have the same in 17.5.0. I think it dates back to when we were not filtering the title field in the LD tables and we “fixed” the Index LT but not the delete pages one. Unless there’s a technical issue to query the titles for deleted docs.

@amilica

There’s no bulk actions or selection capabilities (like restore/delete all,

Pretty dangerous In any case we have protection already about deleting extension pages. I remember we discussed having a config option to fully disable deleting extension pages but I don’t remember if we implemented it. But if we go with batch actions, I believe we should think about it even more.

@amilica

same as for deleted pages, but there’s also no possibility to restore a deleted attachment

Yes, we don’t have this feature. Right now we need to undelete a page to undelete an attachment. See Loading...

@amilica

Even if this linked proposal is around Live Data, UI decisions should be consistent in Live Table too. Obviously, if we plan to migrate all live tables to live data, then there’s no need to change the UI for live tables too.

Note that the Page Index has been moved to LD (It’s using a LD in 17.5.0 for example).

@amilica

If the user didn’t selected anything, the batch actions would be hidden. The Administer bins action will be present regardless of the selection, but with a different style.

I’m not sure about this (I’m actually -0 and probably even -1). Right now we try to have our admins actions in the Admin UI so that they don’t clutter the generic UI. It’s not everyday that you need to administer bins whereas you need to go to the page index a lot more.

Also you didn’t mention it but that button should only appear for users having admin rights ofc.

@amilica

This is how the live data would looks like with the mentioned proposal’s ideas.
Clicking on any of the batch actions available in this screen would make a modal of confirmation appear.

See my comments on that proposal. I don’t like the inconsistency between single actions and batch actions.

Also, IMO “Administer bins” should not be there since it’s not related to the selected items.

@amilica

I like the idea of providing more Admin actions to clean deleted pages and attachments, but I don’t think the Admin UI is the right place for it. We should have this directly in the Page Index UI since when we implement batch actions, we’ll get this feature for free.

@amilica

we compute how much storage would the user get back if they cleaned all deleted items (pages and attachments separately)

Re the computations, would that be done only after you click some button or do we need to always compute it and keep it stored somewhere?

Note that if we have this info, or if it’s easy to compute, I’d actually display it directly in the LD batch action for delete, displayed in parenthesis after the number of items concerned by the batch action.

@amilica

Automatic bin cleaning - we allow the user to set up an automation of cleaning the deleted data based on a set period of time.

Sounds good. This was discussed in the past a few times already.

See also:

• Loading...
• Loading...
• Loading...

Maybe you could also check Loading... and propose something as it seems related?

@amilica

What do you think about all of this?

Thx for the proposal!

I think the proposal is missing the UI screen that would appear on batch restore. For batch delete I assume it’s the one proposed in Batch Actions in Live Data ?

Also, if we do this, we also need to do the same for the other tabs of the Page Index IMO, for consistency.

Thx