Skip to content

Comments

Adds cached sitemap data for sermons#123

Merged
ksigWyatt merged 1 commit intomasterfrom
dev
Feb 6, 2026
Merged

Adds cached sitemap data for sermons#123
ksigWyatt merged 1 commit intomasterfrom
dev

Conversation

@ksigWyatt
Copy link
Member

Introduces a lightweight, cached data feed tailored for sitemap generation. Provides minimal series and message metadata (IDs and dates) via a dedicated endpoint, leveraging existing caching to reduce repeated queries and improve performance. Caches results for two hours to align with sitemap regeneration strategies.

Enhances reliability with comprehensive tests covering cache hits/misses, empty datasets, and partial failures, ensuring consistent behavior and efficient data retrieval.

  • Adds minimal response models for sitemap use
  • Implements service logic to aggregate and cache data
  • Exposes a GET endpoint to retrieve sitemap-ready data
  • Reuses existing caches to minimize backend load
  • Adds unit tests for success, edge cases, and error handling

Introduces a lightweight, cached data feed tailored for sitemap generation. Provides minimal series and message metadata (IDs and dates) via a dedicated endpoint, leveraging existing caching to reduce repeated queries and improve performance. Caches results for two hours to align with sitemap regeneration strategies.

Enhances reliability with comprehensive tests covering cache hits/misses, empty datasets, and partial failures, ensuring consistent behavior and efficient data retrieval.

- Adds minimal response models for sitemap use
- Implements service logic to aggregate and cache data
- Exposes a GET endpoint to retrieve sitemap-ready data
- Reuses existing caches to minimize backend load
- Adds unit tests for success, edge cases, and error handling
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

AI Code Review: No issues found. The changes look good!

@ksigWyatt ksigWyatt merged commit ba06f6c into master Feb 6, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant