Skip to content

fix: set default value for cache store#1308

Merged
mcp97 merged 1 commit into
elizaOS:developfrom
oxSaturn:fix/set-default-cache-store
Dec 21, 2024
Merged

fix: set default value for cache store#1308
mcp97 merged 1 commit into
elizaOS:developfrom
oxSaturn:fix/set-default-cache-store

Conversation

@oxSaturn
Copy link
Copy Markdown
Contributor

Relates to:

Risks

Low.

Background

Seems we introduced this new CACHE_STORE recently, but users with old .env could be missing it, and the error is kind of unclear:

throw new Error(
                `Invalid cache store: ${cacheStore} or required configuration missing.`
            );

What does this PR do?

Set a default value for cacheStore so we won't break the agents with old .env file.

What kind of change is this?

Documentation changes needed?

Testing

Where should a reviewer start?

Detailed testing steps

Copy link
Copy Markdown
Contributor

@mcp97 mcp97 left a comment

Choose a reason for hiding this comment

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

LGTM

@mcp97 mcp97 merged commit 2f56c95 into elizaOS:develop Dec 21, 2024
@oxSaturn oxSaturn deleted the fix/set-default-cache-store branch December 21, 2024 05:41
lalalune pushed a commit that referenced this pull request May 3, 2026
FranceFlapjack pushed a commit to FranceFlapjack/eliza that referenced this pull request May 16, 2026
…tore

fix: set default value for cache store
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.

2 participants