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

Add text to Connection Timed Out alert #743

Conversation

blurfl
Copy link
Collaborator

@blurfl blurfl commented Jul 22, 2018

Address two common issues with the serial connection:
•wrong port selected
•firmware not loaded

Address two common issues with the serial connection:
•wrong port selected
•firmware not loaded
@MaslowCommunityGardenRobot
Copy link
Collaborator

Congratulations on the pull request @blurfl

Now we need to decide as a community if we want to integrate these changes. Vote by giving this comment a thumbs up or a thumbs down. Votes are counted in 48 hours. Ties will not be merged.

I'm just a robot, but I love to see people contributing so I'm going vote thumbs up!

@BarbourSmith
Copy link
Member

Brilliant @blurfl ❤️ 👍 👍 Such a simple change that will save people hours of confusion

@blurfl
Copy link
Collaborator Author

blurfl commented Jul 23, 2018

Improvements or additions to the wording welcome. Are there other causes of the connect/disconnect circle?

@BarbourSmith
Copy link
Member

Those are the two causes I've seen, but let's keep an eye out and if another cause shows up we can add it to the message right away

@MaslowCommunityGardenRobot
Copy link
Collaborator

Time is up and we're ready to merge this pull request. Great work!

@MaslowCommunityGardenRobot MaslowCommunityGardenRobot merged commit 7ad8661 into MaslowCNC:master Jul 24, 2018
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