Skip to content

Memories date fix (again) #588

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.

Sign up for GitHub

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

Merged
JW-CH merged 1 commit into from
Feb 11, 2026
Merged

Memories date fix (again) #588

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,17 +5,18 @@

namespace ImmichFrame.Core.Logic.Pool;

public class MemoryAssetsPool(ImmichApi immichApi, IAccountSettings accountSettings) : CachingApiAssetsPool(new DailyApiCache(), immichApi, accountSettings)

Check warning on line 8 in ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs

View workflow job for this annotation

GitHub Actions / test 

Parameter 'IAccountSettings accountSettings' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.

Check warning on line 8 in ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs

View workflow job for this annotation

GitHub Actions / test 

Parameter 'ImmichApi immichApi' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.

Check warning on line 8 in ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs

View workflow job for this annotation

GitHub Actions / test 

Parameter 'IAccountSettings accountSettings' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.

Check warning on line 8 in ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs

View workflow job for this annotation

GitHub Actions / test 

Parameter 'ImmichApi immichApi' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.

Check warning on line 8 in ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs

View workflow job for this annotation

GitHub Actions / test 

Parameter 'IAccountSettings accountSettings' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.

Check warning on line 8 in ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs

View workflow job for this annotation

GitHub Actions / test 

Parameter 'ImmichApi immichApi' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.

Check warning on line 8 in ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs

View workflow job for this annotation

GitHub Actions / test 

Parameter 'IAccountSettings accountSettings' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.

Check warning on line 8 in ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs

View workflow job for this annotation

GitHub Actions / test 

Parameter 'ImmichApi immichApi' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
{
protected override async Task<IEnumerable<AssetResponseDto>> LoadAssets(CancellationToken ct = default)
{
var memories = await immichApi.SearchMemoriesAsync(DateTime.Today.ToUniversalTime(), null, null, null, ct);
var searchDate = new DateTimeOffset(DateTime.SpecifyKind(DateTime.Today, DateTimeKind.Utc), TimeSpan.Zero);
var memories = await immichApi.SearchMemoriesAsync(searchDate, null, null, null, ct);

var memoryAssets = new List<AssetResponseDto>();
foreach (var memory in memories)
{
var assets = memory.Assets.ToList();
var yearsAgo = DateTime.Now.Year - memory.Data.Year;
var yearsAgo = searchDate.Year - memory.Data.Year;

if (!accountSettings.ShowVideos)
{
Expand Down
Loading