improvement: make limit conditional

This commit is contained in:
Scott Wilson
2024-09-12 16:19:22 -07:00
parent ed3bc8dd27
commit 93f27a7ee8

View File

@@ -131,8 +131,6 @@ export const identityProjectDALFactory = (db: TDbClient) => {
`${TableName.Identity}.id`
)
.where(`${TableName.IdentityProjectMembership}.projectId`, projectId)
.offset(filter.offset ?? 0)
.limit(filter.limit ?? 100)
.orderBy(
`${TableName.Identity}.${filter.orderBy ?? ProjectIdentityOrderBy.Name}`,
filter.orderDirection ?? OrderByDirection.ASC
@@ -140,6 +138,10 @@ export const identityProjectDALFactory = (db: TDbClient) => {
.select(selectAllTableCols(TableName.Identity))
.as(TableName.Identity); // required for subqueries
if (filter.limit) {
void fetchIdentitySubquery.offset(filter.offset ?? 0).limit(filter.limit);
}
const query = (tx || db.replicaNode())(TableName.IdentityProjectMembership)
.where(`${TableName.IdentityProjectMembership}.projectId`, projectId)
.join(TableName.Project, `${TableName.IdentityProjectMembership}.projectId`, `${TableName.Project}.id`)