Skip to content

Conversation

@jr-rk
Copy link

@jr-rk jr-rk commented Oct 30, 2025

Phases MP MM MB MR JM Total
ETA 0 0 0 0 0 0
Developing 0 0 0 0 0 0
Review 0 0 0 0 0 0
Total - - - - - 0
ETA est. 0
ETA cust. - - - - - 0

Problem description

Based on #1059
...Frontend requires cache.msToLive and cache.control settings to function
properly. Excludes server-side cache settings (serverSide, autoSync) while
including client-necessary cache configuration.

image

@jr-rk jr-rk self-assigned this Oct 30, 2025
@jr-rk jr-rk changed the title Add cache config to public config to prevent frontend errors UFAL/Add cache config to public config Oct 30, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds cache configuration properties (msToLive and control) to the public frontend configuration that gets written to config.json. The change enables the frontend to access cache time-to-live settings that are already widely used throughout the application for object caching.

Key changes:

  • Added cache.msToLive and cache.control to the public config object in buildAppConfig function
  • Added inline comment explaining that frontend needs these settings

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@milanmajchrak milanmajchrak merged commit 8de5005 into dtq-dev Oct 31, 2025
4 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.

3 participants