fix: Deleting a collection should not deleted archived documents within

[tommoor]

closes #1775

[tommoor]

@thenanyu make sense to you in terms of desired behavior?

[thenanyu]

This seems like a good patch and worth shipping, but there's a consideration to change the default behavior to archive documents instead of deleting

Then a checkbox can be added in the delete dialog which allows true deletion of included documents

This way we can say something like: "also delete 3 active and 5 archived documents"

[tommoor]

Yea, I like that – default to a soft delete in the future and make this the hard delete option.