Use of timestamp type instead of datetime

[Nosfistis]

I believe having timestamps in the database for row metadata is better than having datetime. Is there any plan to make this an option?

[clarkbw]

I don't have any plans for that, but I'd be happy to take a PR with an option like that.

[Nosfistis]

I don't mind trying to add an option, but I have not delved into creating mixins yet. I will take a try.

[clarkbw]

Ok! Lemme try to get you started.

Change the type which is declared here:

loopback-ds-timestamp-mixin/es6/time-stamp.js

Line 34 in 99abb20

type: Date,

(and here)

loopback-ds-timestamp-mixin/es6/time-stamp.js

Line 40 in 99abb20

type: Date,

To be an option in the options. Also adding that option to this list:

loopback-ds-timestamp-mixin/es6/time-stamp.js

Lines 13 to 19 in 99abb20

	const options = Object.assign({
	createdAt: 'createdAt',
	updatedAt: 'updatedAt',
	required: true,
	validateUpsert: false, // default to turning validation off
	silenceWarnings: false,
	}, bootOptions);

Default this new option to the current value of Date.

Watch out for the defaultFn just in case it does something strange with a timestamp.

Take a look at the current tests and add some when the option changes.

[Nosfistis]

Ok! Thanks for the tips. I'll notify when I have a PR.