dataall
dataall copied to clipboard
Revoking a share item should automatically delete it
After you revoke a share item you still have to delete it. It seems like a completely pointless step. It would be nicer if revoking a share automatically deleted a share item.
@zsaltys thanks for opening the issue!
I have only two considerations against auto-delete
- The customer may need this shares in future for audit, for example. We still can delete it, but we need to make sure, that all important information is saved for the history. I will check now, what exactly we store about data shares after they are deleted. UPD. This info is just deleted from database. So, @dlpzx what so you think, should we save it?
- We should eliminate (or sooth) human error here. May be it's better to add cool-down period after revoking access, e.g. 24 hours and only then delete the share. To complete this idea, may be we should add 'restore access' action, which can be done within 24 hours.
@dlpzx Any thoughts about it?
If the revoke succeeded I am ok deleting the item, I think it simplifies the process. Would this something to be considered for 2.7 @zsaltys ?