neumino / rethinkdbdash

An advanced Node.js driver for RethinkDB with a connection pool, support for streams etc.
MIT License
848 stars 109 forks source link

Rethinkdbdash not respecting datetime fields with timezone #376

Open thelinuxlich opened 6 years ago

thelinuxlich commented 6 years ago

I'm getting datetime fields from grouped queries without their timezones when using the rethinkdbdash driver, I believe there was a similar error in the past? @neumino

thelinuxlich commented 6 years ago

actually anytime I'm returning datetime fields, it prints something like "2018-05-02T17:51:27.758Z", ignoring the timezone, unless I force the datetime field to return with .toISO8601(), is this normal???

neumino commented 6 years ago

Can you provide a table with some data and a query where this doesn't work?

Does it work with the official driver?

thelinuxlich commented 6 years ago

it wasn't rethinkdbdash fault, it would only respect the timezone with the PR I added