zero-ui icon indicating copy to clipboard operation
zero-ui copied to clipboard

dev: remove workaround for deleting the member

Open dec0dOS opened this issue 1 year ago • 4 comments

Deprecate workaround for this: https://github.com/zerotier/ZeroTierOne/issues/859 as It was fixed in https://github.com/zerotier/ZeroTierOne/commit/f8ea7fdc2b734aa3d720fe5c79f7e784c927116f

More info: https://github.com/dec0dOS/zero-ui/blob/559a98850ecec4cc2c39a7720eba39e6aee06572/backend/services/member.js#L173-L192

dec0dOS avatar Oct 08 '23 22:10 dec0dOS

How about this block

https://github.com/dec0dOS/zero-ui/blob/559a98850ecec4cc2c39a7720eba39e6aee06572/backend/routes/member.js#L72-L86

wongsyrone avatar Oct 11 '23 01:10 wongsyrone

@wongsyrone, thanks for pointing! This one is also affected.

dec0dOS avatar Oct 11 '23 01:10 dec0dOS

Another quick thought, my db.json left many zt nodes with deleted set to true, it's better to clean up these leftovers on zero-ui startup. The proposed workflow is to do the member deletion but ignore controller response and delete additional member data and the member entry in db.json.

wongsyrone avatar Oct 11 '23 01:10 wongsyrone

Currently, I'm still thinking for the proper migrations solution for ZeroUI: https://github.com/dec0dOS/zero-ui/issues/75

dec0dOS avatar Oct 11 '23 01:10 dec0dOS