table.timestamps() creates unusable column names with sailsjs

[itsthatguy]

Issue

table.timestamps() creates created_at and updated_at while sailsjs is expecting createdAt and updatedAt

More information: BlueHotDog/sails-migrations#59

[tgriesser]

IMO sql column names shouldn't be camel cased, but if you insist, just use timestamp rather than timestamps. It's one extra line:

table.timestamp('createdAt')
table.timestamp('updatedAt')

[tgriesser]

Also, I'll probably make this configurable at some point but right now I'm refactoring a ton of stuff internally so it's not something I'm going to fix just yet.

[tkrotoff]

@tgriesser Even if you are postponing this, it is best to keep the issue open (and tag it 'backlog', 'future', whatever) - otherwise how will you remember?

[tgriesser]

Eh I'll just leave it open.

[tkrotoff]

Thx :)

[KieranHolroyd]

@tgriesser is this something that could be fixed now, i'm running into similar issues, it just seems like it should have an options object with columnName as an optional attribute or something?

[kibertoad]

@KieranHolroyd This will be fixed in 1.0.0 which should be released this week. There is a new option for that.

[OlivierCavadenti]

Yes, this function have now useCamelCase option:

timestamps — table.timestamps([useTimestamps], [defaultToNow], [useCamelCase])