Skip to content
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

feat: add sentinel support #59

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: add sentinel support #59

wants to merge 1 commit into from

Conversation

cyrinux
Copy link

@cyrinux cyrinux commented Nov 16, 2024

This is an attempt to support redis-sentinel mode. 😄

@f00wl
Copy link

f00wl commented Nov 16, 2024

Interested to have sentinel support!

@underyx
Copy link
Owner

underyx commented Nov 27, 2024

Thank you for opening this PR! I don't currently have any projects that use flask-redis so it's a bit hard for me to test it. Would you like to be added as a maintainer to this project?

@cyrinux
Copy link
Author

cyrinux commented Nov 28, 2024

Hi @underyx , thanks for the proposition but on my side i'm not a good python dev nor a daily flask user, I did this for one particular project, but I'm afraid to not be the good guy for maintain this properly.

Can you maybe open an issue about the fact that you need more maintainer ? 🙏🏻

@f00wl in case you or know people who can take this cap 🙏🏻 feel free to tell us !

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