dynamodb-toolbox icon indicating copy to clipboard operation
dynamodb-toolbox copied to clipboard

Add inverse transform

Open GeertWille opened this issue 3 years ago • 9 comments

We needed a way to be able to transform a value coming from DynamoDB before parsing it. Thats why we added an inverseTransform field option.

GeertWille avatar Oct 28 '20 20:10 GeertWille

Sorry for the delay @GeertWille. I've been working on the TypeScript conversion.

This addition makes sense, but I'm not sure inverseTransform is the right label. Maybe format would be better? I'll incorporate this into the v0.3 branch as that will soon be merged to master.

jeremydaly avatar Nov 19 '20 13:11 jeremydaly

Hi @GeertWille, I just released v0.3, but didn't get a chance to get this in. I'm planning on implementing this soon.

jeremydaly avatar Dec 12 '20 23:12 jeremydaly

Hey @jeremydaly, No problem. Let me know if I can do anything to help.

GeertWille avatar Dec 12 '20 23:12 GeertWille

Hi @jeremydaly and @GeertWille any news on this one?

It would be helpful for us as well. We need to get date objects back from created/modified etc. With inverseTransform or format we can wrap it in a new Date() function. Thanks!

BabyDino avatar Mar 31 '21 08:03 BabyDino

Just for date manipulations, this feature would be very nice! Is it plan on next release? Thanks for your hard work!

Wingjam avatar Sep 09 '21 14:09 Wingjam

@BabyDino @Wingjam Not sure, didn't hear back from @jeremydaly yet so I'm still using my own fork 😞

GeertWille avatar Sep 30 '21 08:09 GeertWille

@jeremydaly Any updates on this? We'd love to see this feature implemented as it would help us tremendously.

Namstel avatar Nov 09 '21 13:11 Namstel

@jeremydaly Any updates on this?

brahimmouhamou avatar Jun 22 '22 07:06 brahimmouhamou

Hey @GeertWille, first of all, thanks for creating this PR, you're awesome 😎

I agree with @jeremydaly about the naming.

Can you rebase the changes to the latest version? also, let's remove the unrelated changes and discuss them separately 🙏

naorpeled avatar Aug 05 '22 23:08 naorpeled

Closing this one in favor of #332

GeertWille avatar Sep 20 '22 08:09 GeertWille