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

Improve synchronisation of the pairing code message. #10

Merged
merged 2 commits into from
Jul 1, 2018

Conversation

ensonic
Copy link
Contributor

@ensonic ensonic commented Jun 26, 2018

We did send the code to the server and showed the code at the same time.
This will let the user enter the code before we managed to connect. Now
we show a "connecting..." message first and only show the code when we
conencted.
See OpenRoberta/robertalab-ev3dev#30

@ensonic
Copy link
Contributor Author

ensonic commented Jun 26, 2018

If this sounds good. I'll send another one for stretch.
On my openrobertalab package I will need to change the behavior depending on whether this has been merged (and released) or not. Right now 'brickman --version` does not work. Would you mind if I send a PR that adds this?

@dlech
Copy link
Member

dlech commented Jun 26, 2018

This looks fine. FYI, I wasn't planning on doing any more ev3dev-jessie releases. We can still update the brickman package though if people are using it for OpenRoberta.

Adding a --version command line option is OK too.

We did send the code to the server and showed the code at the same time.
This will let the user enter the code before we managed to connect. Now
we show a "connecting..." message first and only show the code when we
conencted.
See OpenRoberta/robertalab-ev3dev#30
This can be used to quickly check the version :)
@ensonic
Copy link
Contributor Author

ensonic commented Jun 28, 2018

$ /usr/sbin/brickman --version
brickman v0.8.1

I'll prepare another pull request for stretch.

@dlech dlech merged commit 527dae6 into ev3dev:ev3dev-jessie Jul 1, 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.

2 participants