magda-io / magda

A federated, open-source data catalog for all your big data and small data
https://magda.io
Apache License 2.0
505 stars 92 forks source link

Should set the default `order by` field for API /auth/roles/:roleId/permissions #3424

Closed t83714 closed 1 year ago

t83714 commented 1 year ago

Should set the default order by field for API /auth/roles/:roleId/permissions

When not setting the default order by field in SQL, API like /auth/roles/:roleId/permissions might return records in different order on different pages.

We should always specific orderBy clause (default order by primary key id)

t83714 commented 1 year ago

closed via PR: https://github.com/magda-io/magda/pull/3423