[FR] Delete multiple locations at once

[francescocarzaniga]

Please verify that this feature request has NOT been suggested before.

• [x] I checked and didn't find a similar feature request

Problem statement

There is currently no option to delete multiple locations at once. It is possible to select multiple locations in the menu, but the only option is to set their parent.

Suggested solution

A menu item to delete all selected locations, and optionally their children and their items (as in the single location delete menu).

Describe alternatives you've considered

It is currently possible to do this by creating a temporary parent location, moving all locations to said parent, and deleting the parent together with the children.

Examples of other systems

No response

Do you want to develop this?

• [ ] I want to develop this.

[SchrodingersGat]

I would advise against implementing such a feature. Deleting a stock location needs to be carefully considered, as it is part of a tree structure and a determination needs to be made as to what happens to stock items or stock locations which exist in (or below) the location.

I think that it is an action that should be taken individually and carefully considered

[francescocarzaniga]

I understand what you mean but this seems largely a non-issue. A simple modal could be implemented a-la "Write DELETE to confirm deletion" if this is a significant worry. If I want to get rid of like 10+ locations it's not very ergonomic to do it one by one...

Also, the API already exists and acknowledges precisely my point, so it's mostly a question of adding the frontend interface.

[matmair]

I tend to lean on @SchrodingersGat side here; Would we accept a PR or is this a wontfix?

[SchrodingersGat]

If @francescocarzaniga can provide a clean implementation I'd review it - but it does need to be treated carefully as deleting locations is more complex than other items without tree dependencies