-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sequelize delete property of meta object error #234
Comments
This rather looks like a problem with the sequelize package? |
It is, sequelize does not support v3 of this connector yet. Almost all tests of the sequelize package pass on 3.0.2 but more fail on 3.1.0 due to the non-enumerable meta. The advice of the sequelize team is to not upgrade to v3 of the mariadb connector until v7 of sequelize |
do you have some link @WikiRik / @knoxcard2 ? correction is easy to do, and i can make a PR |
You can check out which tests are failing in this renovate PR; sequelize/sequelize#15311 |
|
Ah yes, I forgot this issue was open as well. Thanks for posting it here! |
Until v3 of the connector is supported: npm install --save mariadb@2 |
it is supported now ? |
/etc/nginx/domains/buycanna.io/node_modules/sequelize/lib/dialects/mariadb/query.js:110
delete data.meta;
TypeError: Cannot delete property 'meta' of [object Array]
at Query.formatResults (/etc/nginx/domains/buycanna.io/node_modules/sequelize/lib/dialects/mariadb/query.js:110:7)
at Query.run (/etc/nginx/domains/buycanna.io/node_modules/sequelize/lib/dialects/mariadb/query.js:73:17)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async /etc/nginx/domains/buycanna.io/node_modules/sequelize/lib/sequelize.js:314:16
Node.js v19.6.1
The text was updated successfully, but these errors were encountered: