Mick Hansen

That syntax is not currently supported (except with `separate: true` which only works for hasMany atm). You need to use the slighter longer/sweird syntax: ``` findAll({ ... order: [ [models.Image,...

Ah yes if you have an alias on your incldue you'll need to use `[{ model: models.Image, as: 'IdeaHeaderImages' }, 'updated_at', 'asc']`

Odd, that should simply work.

There's a confirmed issue with `through.attributes`

Hmm, but the value of `tableName` could be a JSON object that needs to be inserted as JSON. We probably want to use `$` prefixed keys.

This change would also not make a lot of room for introducing proper prepared queries later, perhaps a different identifier than `:` for tables/identifiers? Not sure what other libraries do...

@eraye1 Mostly just not using table as a replacement. If you have a dynamic table you can use our private quoteTable manually.

No immediate plans. Syntax should definitely be equal to include. Should be simple enough after https://github.com/sequelize/sequelize/pull/6170 is merged

`book.setGenres` should support `[1, 2, 3]`. The rest is stuff we'd like to do by expanding nested create/update stuff.

http://docs.sequelizejs.com/en/latest/api/associations/has-many/ `setGenres([instances], {/*options*/ transaction: t}`