Capitalisk / ldpos-pg-dal

Knex Data Access Layer for LDPoS
MIT License
2 stars 0 forks source link

Cleanup upsert logic #12

Open jondubois opened 2 years ago

jondubois commented 2 years ago

Remove string replace operations:

https://github.com/Capitalisk/ldpos-pg-dal/blob/555cff6ff51b8098af9f37ee7f5aaaef3159cf73/knex/knex-client.js#L69-L77

jondubois commented 2 years ago

Also, the 4th parameter doesn't appear to be used so it may be better to remove it.