-
-
Notifications
You must be signed in to change notification settings - Fork 127
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: add prefixes to Redis #137
Conversation
I don't know what happened with the |
bc29ae5
to
989b41b
Compare
I'll check this week. |
989b41b
to
caaa740
Compare
@@ -101,7 +101,6 @@ class Cart extends React.Component | |||
} | |||
|
|||
$.post(this.props.addToCartURL, { | |||
date: this.props.deliveryDate, |
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.
Is this normal?
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.
Yep (I dont remember why this change is here though)
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.
I extracted it into a single commit 😉
caaa740
to
de0d745
Compare
de0d745
to
95678b0
Compare
95678b0
to
b8bcac8
Compare
b8bcac8
to
4982118
Compare
F*x #63
I think we just need to redeploy each instance and it will be fine, as data are always polled from Postgres.
Note that I did a small trick to prefix
subscribe
calls.Please don't merge I have still a question about this change