mickhansen / graphql-sequelize

GraphQL & Relay for MySQL & Postgres via Sequelize
MIT License
1.9k stars 172 forks source link

OrderBy doesn't pass multiple to the query #716

Closed intellix closed 1 year ago

intellix commented 1 year ago

Example query where only CREATED_AT_DESC and ID_DESC (by default) are used:

users(orderBy: [CREATED_AT_DESC, TOTAL_DEPOSIT_DESC]) {

with orderBys defined like so:

UserOrderBy: {
  ID_DESC: ['id', 'DESC'],
  CREATED_AT_DESC: ['createdAt', 'DESC'],
  LAST_DEPOSIT_AT_DESC: ['lastDepositAt', 'DESC'],
}

I have the fix already in a patch-package but just never upstreamed it. Leaving this ticket here to remind me to add a PR with it later

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.