-
Notifications
You must be signed in to change notification settings - Fork 13
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
Bothan.io Bothan Deploy hangs on name with <=2 chars #201
Comments
repeated auto deploy and now it works fine. Seeing as I didn't capture any logs at the time this happened it's going to be hard to determine what (if anything) has changed, so closing this for now |
Oh, that's very odd. |
@agentGav was experiencing this as well. Gavin, do you mind retrying the bothan deployment as it's working here OK now? |
Is it worth getting Airbrake here so we can track fails? |
it works as long as the name is >= 3 chars
spins on 2 chars (think it's a limit on heroku)
…On 27 April 2017 at 12:48, James Smith ***@***.***> wrote:
@agentGav <https://github.com/agentGav> was experiencing this as well.
Gavin, do you mind retrying the bothan deployment as it's working here OK
now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#201 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABVvuxGVu3wUlgwnD1uw1aEcp9jsec_Rks5r0ICRgaJpZM4M1rxL>
.
|
AHA, ok. Trust you to find a degenerate corner case. We can add validation to the input field to make sure it's >=3 chars. |
So I literally can't replicate this because so many 3 char combinations appear to trigger |
I think the issue is with less than 3, so the validation should be enough. |
following the bothan deploy from bothan.io to heroku currently hangs and does not create the app
The text was updated successfully, but these errors were encountered: