Rik Smale
Rik Smale
We have various tests in https://github.com/sequelize/sequelize/tree/v6/test/types that use `CreationOptional`. Can you make a PR adding a failing test?
Can you reference the part from the manual you are referring to? In my opinion it's the correct behaviour that sequelize does not assume which schema you are using the...
That's the v7 docs for using `@sequelize/core` alphas fyi. But the v6 API reference says the same; https://sequelize.org/api/v6/class/src/sequelize.js~sequelize#instance-method-sync . I'll take a look later today to see what the code...
> This seems to me that using different schemas with MS SQL (at least) is broken. That is possible, we've improved our support for schemas quite a bit in the...
Okay, I looked into it a little bit. These are my notes but no conclusion on how to fix it. --- The issue is in `Model.sync()`. The following code already...
The status is that the linter and unit tests are failing. Until those are fixed we won't review. But we can assist if you have any questions about it
Quick first thoughts: - We as maintainers already identified that `describeTable` needs to be improved, at least so that results are uniform across dialects. See this comment from last year;...
What I can remember from earlier discussions with the team is that we want to reduce the amount of logic in the QueryInterface so that QueryGenerator functions can be used...
Can you share a bit more on why this is better. Do you have an example of where it causes unexpected behaviour as you mentioned? This behaviour also seems to...
@joker00777 can you take a look at this?