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

src/tryit.jade: Remove cookie #4

Closed
wants to merge 1 commit into from
Closed

src/tryit.jade: Remove cookie #4

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 28, 2017

Many pump nodes are down. If someone clicks Try It and gets sent to an offline node they might want to go back and Try It again. Not realizing that a cookie has been sent they keep going back to the same place over and over. Head, meet wall. Hopefully removing the cookie means they can potentially be sent to a different node.

Many pump nodes are down. If someone clicks Try It and gets sent to an offline node they might want to go back and Try It again. Not realizing that a cookie has been sent they keep going back to the same place over and over. Head, meet wall. Hopefully removing the cookie means they can potentially be sent to a different node.
@ghost ghost mentioned this pull request Jan 28, 2017
@strugee
Copy link
Member

strugee commented Jan 28, 2017

@LuckyPigeon did you test this? AFAICT this will completely break Try It

@ghost ghost closed this Jan 31, 2017
@strugee
Copy link
Member

strugee commented Feb 1, 2017

:(

@LuckyPidgeon if you ever want to come back and try again, we'd love to have you.

This pull request was closed.
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