From f37c19cc846c42d235ca994508fc52f6594733b9 Mon Sep 17 00:00:00 2001 From: Varun Raj Date: Sat, 11 Jul 2026 04:26:47 +0000 Subject: [PATCH] fix: add album_user.role to GROUP BY in albums list query The myRole select added for issue #286 wasn't included in the GROUP BY clause, causing a 500 ("column album_user.role must appear in the GROUP BY clause") whenever includeShared=true was requested. --- src/pages/api/albums/list.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/pages/api/albums/list.ts b/src/pages/api/albums/list.ts index 50f5d6d..bd94d6d 100644 --- a/src/pages/api/albums/list.ts +++ b/src/pages/api/albums/list.ts @@ -85,7 +85,7 @@ export default async function handler( .leftJoin(assetFaces, eq(assets.id, assetFaces.assetId)) .leftJoin(person, and(eq(assetFaces.personId, person.id), eq(person.isHidden, false))) .where(eq(albumUsers.userId, currentUser.id)) - .groupBy(albums.id, albumUsers.userId) + .groupBy(albums.id, albumUsers.userId, albumUsers.role) .orderBy(desc(albums.createdAt)); const sortedAlbums = sortAlbums(dbAlbums as IAlbum[], sortBy, sortOrder);