Support for JSONB about sequelize-paper-trail

[jdgarvey]

I am running into this as well, I am guessing that paper trail does not support JSON or JSONB?

[nielsgl]

AFAIK it doesn't support it but I'm happy to merge a pull request for this.

[nielsgl]

@g00ds0n @jdgarvey JSONB should be supported with the latest version 3.0.0-rc.11. Let me know whether it does or does not work :)

[g00ds0n]

Nope I still don't see my JSONB objects inside of document. If you tell me where to start I can try and make a PR for you to look at.

[g00ds0n]

@nielsgl As far as I can tell this is omitting JSONB columns because they are objects. I'm not exactly sure how to fix this.

sequelize-paper-trail/lib/index.js

Lines 361 to 367 in 283b8b7

	currentVersion = _.omitBy(
	currentVersion,
	i =>
	i != null &&
	typeof i === 'object' &&
	!(i instanceof Date),
	);

[g00ds0n]

I don't fully understand what that omitBy statement does, but I think using rawAttributes might fix this:

currentVersion = _.pick(currentVersion, Object.keys(instance.rawAttributes));

[g00ds0n]

Check PR #85.

[yujiosaka]

fixed it in my fork
yujiosaka@d22cd54