radash icon indicating copy to clipboard operation
radash copied to clipboard

[FEATURE REQUEST] .shakeDeep

Open mstykow opened this issue 2 years ago • 3 comments

It would be awesome if there was a version of the shake method which also drops unwanted values in a nested object. The current implementation will leave the following object unchanged:

{ a: { b: undefined }}

mstykow avatar Feb 08 '23 10:02 mstykow

Very interesting, I can see the value for sure. I would be down to either create another function and call it shakeDeep or add deep as an argument to the existing shake function. Either way.

If anyone wants to PR this 🍻

sodiray avatar Feb 23 '23 04:02 sodiray

Done in #261.

mstykow avatar Feb 25 '23 11:02 mstykow

I would love to see this feature, maybe we can make it optional?

I personally was under the wrong assumption it was acting in this way already

MatteoMaso avatar Nov 24 '23 15:11 MatteoMaso

Hey @mstykow and @MatteoMaso! I and some others are maintaining a fork of Radash called Radashi. We would gladly accept a PR for this feature.

aleclarson avatar Dec 19 '24 18:12 aleclarson