Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: migrate library repository to kysely #15271

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

danieldietzler
Copy link
Member

No description provided.

@danieldietzler danieldietzler force-pushed the refactor/kysely-library branch 2 times, most recently from 26b3d5a to 48790d0 Compare January 11, 2025 22:53
server/src/repositories/library.repository.ts Outdated Show resolved Hide resolved
server/src/repositories/library.repository.ts Outdated Show resolved Hide resolved
server/src/repositories/library.repository.ts Show resolved Hide resolved
server/src/repositories/library.repository.ts Outdated Show resolved Hide resolved
@danieldietzler danieldietzler force-pushed the refactor/kysely-library branch from 48790d0 to 9415b57 Compare January 13, 2025 22:01
server/src/interfaces/library.interface.ts Outdated Show resolved Hide resolved
create(library: Insertable<Libraries>): Promise<LibraryEntity> {
return this.db
.insertInto('libraries')
.values(library as LibraryEntity)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the cast here?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left over from an earlier method signature, ty!

@danieldietzler danieldietzler force-pushed the refactor/kysely-library branch 2 times, most recently from 520d6e3 to a910de3 Compare January 14, 2025 20:03
@danieldietzler danieldietzler force-pushed the refactor/kysely-library branch from a910de3 to ee4775b Compare January 14, 2025 20:07
@alextran1502 alextran1502 merged commit a2207f2 into main Jan 15, 2025
36 checks passed
@alextran1502 alextran1502 deleted the refactor/kysely-library branch January 15, 2025 21:01
vladd11 pushed a commit to vladd11/immich that referenced this pull request Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants