-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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.
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
fix: redis url to not be optional anymore #7850
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This pull request moves the REDIS_URL environment variable from optional to required in the twenty-server package's .env.example file.
- Relocated REDIS_URL from optional to required section in
/packages/twenty-server/.env.example
- Change addresses issue [BUG] REDIS_URL is not optional in twenty-server anymore #7849, ensuring database reset functionality works properly
- May impact existing setups without Redis configured
- Reflects that REDIS_URL is now mandatory for the application's operation
- Improves clarity for new users setting up the environment
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile
Log
|
@Nabhag8848 Thank you |
/award 150 |
Awarding Nabhag8848: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/Nabhag8848 |
Description
REDIS_URL
is requiredCloses [BUG] REDIS_URL is not optional in twenty-server anymore #7849
Might be related When deploying the application in Docker (locally, on cloud Linux, or in Render), both the worker and server containers fail to connect to Redis, resulting in an
ECONNREFUSED
error. #7768