orderBy is deprecated

[uhthomas]

I have searched the existing issues, both open and closed, to make sure this is not a duplicate report.

• Yes

The bug

Noticed this in the logs whilst doing some dev.

immich_server            | `orderBy(..., expr)` is deprecated. Use `orderBy(..., 'asc')` or `orderBy(..., (ob) => ...)` instead.

The OS that Immich Server is running on

N/A

Version of Immich Server

0986a71

Version of Immich Mobile App

N/A

Platform with the issue

• Server
• Web
• Mobile

Your docker-compose.yml content

N/A

Your .env content

N/A

Reproduction steps

N/A

Relevant log output

Additional information

No response

[uhthomas]

immich/server/src/repositories/audit.repository.ts

Line 32 in 0986a71

.orderBy(['audit.entityId desc', 'audit.entityType desc', 'audit.createdAt desc'])

immich/server/src/repositories/person.repository.ts

Line 195 in 0986a71

.orderBy(sql`NULLIF(person.name, '')`, sql`asc nulls last`)

immich/server/src/repositories/sync.repository.ts

Line 162 in 0986a71

.orderBy(['id asc']) as SelectQueryBuilder<DB, T, D>;

immich/server/src/repositories/sync.repository.ts

Line 173 in 0986a71

.orderBy(['updateId asc']) as SelectQueryBuilder<DB, T, D>;

immich/server/src/repositories/tag.repository.ts

Line 71 in 0986a71

return this.db.selectFrom('tags').select(columns.tag).where('userId', '=', userId).orderBy('value asc').execute();

[jrasm91]

I believe these have all been fixed now.