Migrate ImmichLogger over to injected ILoggerRepository #2604

New issue

Closed

opened 2026-02-20 01:14:30 -05:00 by deekerman · 2 comments

deekerman commented

2026-02-20 01:14:30 -05:00

Owner

Originally created by @bo0tzz on GitHub (Apr 15, 2024).

Originally assigned to: @iamamansharma on GitHub.

#8823 introduces a new ILoggerRepository. All remaining usages of ImmichLogger should be migrated to use this new injected logger.

Originally created by @bo0tzz on GitHub (Apr 15, 2024). Originally assigned to: @iamamansharma on GitHub. #8823 introduces a new ILoggerRepository. All remaining usages of ImmichLogger should be migrated to use this new injected logger.

deekerman

2026-02-20 01:14:30 -05:00

closed this issue
added the
🗄️server

good first issue

changelog:skip
labels

deekerman commented

2026-02-20 01:14:31 -05:00

Author

Owner

@iamamansharma commented on GitHub (Apr 16, 2024):

can you assign this to me?

@iamamansharma commented on GitHub (Apr 16, 2024): can you assign this to me?

deekerman commented

2026-02-20 01:14:31 -05:00

Author

Owner

@jrasm91 commented on GitHub (Apr 16, 2024):

#7156 updates unit tests to use vitest, so make sure you start on this after that is merged (if you want to avoid merge conflicts)

@jrasm91 commented on GitHub (Apr 16, 2024): #7156 updates unit tests to use vitest, so make sure you start on this after that is merged (if you want to avoid merge conflicts)

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/immich#2604

No description provided.

Rows
Columns