confuse icon indicating copy to clipboard operation
confuse copied to clipboard

allow removing items or sources from config

Open cloud-rocket opened this issue 3 years ago • 2 comments

@sampsyo - are you open to merge something like this to allow removing sources and deleting config items.

(This branch is for a review and not for merging now as it does not contain enough testing).

Thanks

cloud-rocket avatar May 14 '21 19:05 cloud-rocket

@sampsyo - do you think you can merge it?

Thank you!

cloud-rocket avatar Oct 25 '21 20:10 cloud-rocket

Hi! Seems generally useful, but perhaps somewhat tricky to explain exactly what's going on. Removing entire sources seems like it could have somewhat unpredictable effects on the data model. Removing individual keys (i.e., the opposite of add) seems perhaps more intuitive, but it also seems like it might be sort of difficult to guarantee that add and remove are actually opposites—that is, that removing something that was previously added at a given point in the configuration tree leaves us back in the original place. Does that make any sense?

It might be useful to have a somewhat broader conversation about the use case you're running into here so we can refine the proposal a bit. Is there a specific case where this came up?

sampsyo avatar Oct 26 '21 18:10 sampsyo