Override the updated_at

[krichards]

How can I override the updated at when adding and updating rows.

I need to sync with another data source and set the updated at under some circumstances.

I tried adding including the updated_at but it complains that there are multiple instances of the same column.

[romeerez]

Could you provide a code example that is causing the error?

Simply providing the updated_at key-value to the update method should work just fine, so there should be something else that is causing the trouble.

[romeerez]

Here is the test to make sure that when you provide your own value for the updatedAt it works correctly.

But I can see the problem, it's about using a snake cased updated_at.

JS conventions tell us to use camelCase in the code; the ORM is designed with this in mind.

By default, columns in the app code are named in camelCase and db columns are expected to be in camelCase.
You can set snakeCase: true (here are the docs for it), columns in the app code are still in camelCase, but they are translated to snake_case when producing SQL.

If you try to do this:

const q = table.where().update({ updated_at: "1234"});

You should have TS error saying that updated_at is not allowed, so if you are doing this and not getting the TS error I'd like to see your code to figure out what went wrong.

[krichards]

I think the problem is that the key specified on the snake timestamps is updated_at, whilst the actual key used on the update object will be 'updatedAt' so it is not removed and you get the duplication error.

If you try and use 'updated_at' it's filtered out because it's not a valid column name and so the default behaviour remains.

[krichards]

I've made this change and then my test (using snake case and updatedAt) passes.

[krichards]

@romeerez I've created a PR containing the test and what I believe is the fix. You should see if the fix is removed you get duplicate updated_at in the SQL.

#144

[romeerez]

Thank you a lot for spending time and making a PR! I'll re-check it in the near days, sorry that don't have much time right now, and will merge