-
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
Adding instruction for running redis #7434
Adding instruction for running redis #7434
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 PR enhances the local setup documentation by adding instructions for starting the Redis server, a crucial step for optimal performance of the Twenty application.
- Added step-by-step instructions for starting Redis server on Linux, Mac OS, and Windows (WSL) in
packages/twenty-website/src/content/developers/local-setup.mdx
- Inserted the Redis server startup step before the database setup and application launch steps
- Improved overall clarity of the setup process by explicitly mentioning the need for a running Redis cache
1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings
```bash | ||
redis-server.exe | ||
``` |
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.
logic: Windows Redis start command is incorrect. Redis on Windows is not officially supported and requires different setup.
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.
To be honest, I don't have windows. So it googled a way to run redis on windows but it may be wrong !
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.
@guillim Twenty doesn't run directly on Windows but on WSL, please fix this accordingly to WSL setup
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.
Good point @BOHEUS thanks!
12849b3
to
d01ddeb
Compare
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.
Thanks for your first contribution 🙌
fixes twentyhq#7433 --------- Co-authored-by: Félix Malfait <[email protected]>
fixes #7433