Open artis3n opened 7 years ago
The recent addition of ON CONFLICT in #335 uses in a test the format:
post.insert({ + content: 'test', + userId: 2 + }).onConflict({ + columns: ['userId'], + update: ['content'] + }),
However, if a SQL definition is constructed as:
sql.define({ name: 'test', columns: [ { name: 'start_time' } ], snakeToCamel: true, });
Then, the ON CONFLICT clause will not respect the snakeToCamel setting, and will require update: ['start_time'] instead of update: ['startTime'].
snakeToCamel
update: ['start_time']
update: ['startTime']
This should be noted somewhere or modified to support the snakeToCamel property.
Hi, Thanks for reporting, I will share an update soon.
@brianc @arikalfus issue is resolved by #342
The recent addition of ON CONFLICT in #335 uses in a test the format:
However, if a SQL definition is constructed as:
Then, the ON CONFLICT clause will not respect the
snakeToCamel
setting, and will requireupdate: ['start_time']
instead ofupdate: ['startTime']
.This should be noted somewhere or modified to support the
snakeToCamel
property.