LedgerSMB icon indicating copy to clipboard operation
LedgerSMB copied to clipboard

Delete attached files for Transactions and Invoices

Open aungzwin10 opened this issue 4 years ago • 3 comments

According to our discussion in the channel, system lack of deleting attached files. So need to add that feature.

Add a new column 'Delete' after Attached By and delete button for each line OR checkbox for each line and delete button beside [Attach]

08DB6768-12DE-4598-BA05-FDD24BD8C8E6_1_105_c

aungzwin10 avatar Sep 24 '21 08:09 aungzwin10

Should anybody be able to do this? Or should it be restricted? If restricted, should it be subject to review by a second person?

ehuelsmann avatar Jan 16 '23 10:01 ehuelsmann

We have a workflow to show the history of the action done on the invoice, right? I think, anyone who can edit the invoice can also edit the attached files.And the workflow history show the action done with the username is enough.

aungzwin10 avatar Jan 16 '23 11:01 aungzwin10

We have a few entries that look like this as well - with multiple attachments because someone uploaded the wrong one the first tiime. As aungzwin says if you have the permission to edit an invoice that same set of permissions should be enough to let you delete an attachement. (Ideally with that noted in the workflow history)

tonka128 avatar Jan 17 '23 08:01 tonka128