fix: Memories date

[3rob3]

Closes #631

Summary by CodeRabbit

• Bug Fixes
  • Improved memory search date calculation to properly handle current time and timezone information, replacing the previous UTC-based timestamp approach. This ensures more accurate date-based memory retrieval and anniversary date calculations.

[coderabbitai]

📝 Walkthrough

Walkthrough

Updated MemoryAssetsPool.LoadAssets to compute searchDate using DateTimeOffset.Now instead of constructing a UTC midnight timestamp from DateTime.Today. This modifies the timezone basis and time boundary used for the SearchMemoriesAsync call and affects the yearsAgo calculation.

Changes

Cohort / File(s)	Summary
Date Calculation Update ImmichFrame.Core/Logic/Pool/MemoryAssetsPool.cs	Changed searchDate computation from UTC midnight timestamp to DateTimeOffset.Now, affecting timezone basis and time offset for memories search and year calculation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

• immichFrame/ImmichFrame#492: Modifies the date value passed to SearchMemoriesAsync in the same method.
• immichFrame/ImmichFrame#588: Replaces the searchDate source calculation and adjusts year calculations in MemoryAssetsPool.LoadAssets.

Suggested labels

fix

Poem

🐰 A moment in time, now captured just right,
From midnight UTC to the present so bright,
DateTimeOffset takes the stage with a bound,
Memories will search where the real time is found!

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title is vague and generic, using non-descriptive terms that don't clearly convey what the date-related fix entails or why it matters.	Consider revising to something more specific like 'fix: Use DateTimeOffset.Now for memory search date calculation' to clearly indicate the nature of the fix.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch dev.3rob3.memories

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}