[*] DateOnly - Sequelize 4 returns date strings not timestamps so don't shift about forest-express-sequelize HOT 8 CLOSED

Hi @arnaudbesnier the column type was date

from forest-express-sequelize.

Any idea on an ETA for reviewing this one? I'd like to get us off our branch, but getting this resolved will be necessary for that to happen. Thanks!

from forest-express-sequelize.

Hi @gcoombe, I cannot reproduce your issue in my postgresql/sequelize4 environment with a date type.

fieldDateOnly: { type: DataTypes.DATEONLY },

Could you share with us your database type, the column type you have configured and the field declaration in Sequelize?

Thanks

from forest-express-sequelize.

FYI here are my fix versions:

• ForestAdmin/forest-express#175
• #189

I need your response to find a way to reproduce and be be confident with the patch. Thank you.

from forest-express-sequelize.

Hi @gcoombe, can you please help?

from forest-express-sequelize.

Hi @arnaudbesnier apologies for the delay. The noted fix version seem to fix the problems on my end. I can still repro the issue on master. My set up is as follows:

• column type: : DataTypes.DATEONLY
• db: postgres
• sequelize version: 4.37.1

Your server must be in a location with a negative UTC shift to repro.

from forest-express-sequelize.

Thanks @gcoombe, I think you missed the most important part of the response, which is the column type defined in your Postgres database. ;)

I tried to reproduce the issue with the exact same conditions, so I guess the difference we have is the column type.

from forest-express-sequelize.

Hi @gcoombe, the fix has been released in Forest v2.12.1.

Thanks for the feedback.

🌲🌲🌲

from forest-express-sequelize.