search

mickhansen / graphql-sequelize

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

fix: add default scope order #674

Closed Jekiwijaya closed 4 years ago

Jekiwijaya commented 5 years ago

Current behavior override the defaultScope order, this commit fix by taking the defaultScope order first then add the default order by id.

Closed #488

stale[bot] commented 4 years 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.

stale[bot] commented 4 years 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.

mickhansen commented 4 years ago

PR still looks good, just needs a test

stale[bot] commented 4 years 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.