mirror of https://github.com/buster-so/buster.git
Add consistent ordering to individual_permissions queries
- Add .order_by(users::email) to individual_permissions_query in dashboard handler - Add .order_by(users::email) to individual_permissions_query in metrics handler - Add .order_by(users::email) to individual_permissions_query in collections handler - Ensures consistent alphabetical ordering by email across all endpoints Fixes BUS-1477 Co-Authored-By: nate@buster.so <nate@buster.so>
This commit is contained in:
parent
a7e023ff6f
commit
5082831630
|
@ -133,6 +133,7 @@ pub async fn get_collection_handler(
|
|||
users::name,
|
||||
users::avatar_url,
|
||||
))
|
||||
.order_by(users::email)
|
||||
.load::<AssetPermissionInfo>(&mut conn)
|
||||
.await;
|
||||
|
||||
|
|
|
@ -346,6 +346,7 @@ pub async fn get_dashboard_handler(
|
|||
.filter(asset_permissions::identity_type.eq(IdentityType::User))
|
||||
.filter(asset_permissions::deleted_at.is_null())
|
||||
.select((asset_permissions::role, users::email, users::name, users::avatar_url))
|
||||
.order_by(users::email)
|
||||
.load::<AssetPermissionInfo>(&mut conn)
|
||||
.await;
|
||||
|
||||
|
|
|
@ -399,6 +399,7 @@ pub async fn get_metric_handler(
|
|||
.filter(asset_permissions::identity_type.eq(IdentityType::User))
|
||||
.filter(asset_permissions::deleted_at.is_null())
|
||||
.select((asset_permissions::role, users::email, users::name, users::avatar_url))
|
||||
.order_by(users::email)
|
||||
.load::<AssetPermissionInfo>(&mut conn)
|
||||
.await;
|
||||
|
||||
|
|
Loading…
Reference in New Issue