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

[Control] Init gui #186

Closed
wants to merge 2 commits into from
Closed

[Control] Init gui #186

wants to merge 2 commits into from

Conversation

ronanM
Copy link
Contributor

@ronanM ronanM commented Apr 21, 2020

Some Control GUI code from a Rust beginner.

@eliottvincent
Copy link
Member

Hello there! Thanks for your PR, it's deeply appreciated as we are in the process of building the control GUI as I write this comment. @valeriansaliou and @BlackYoup will take a look at it 😊

@eliottvincent eliottvincent requested a review from BlackYoup April 22, 2020 13:15
@BlackYoup
Copy link
Collaborator

@ronanM Hello! Sorry for the wait here and thank you very much for this PR.

We've been moving on that Control UI part those last few days with some code pushed this afternoon and a new pull request opened to add a few more features (#194). Some of the changes you made are nice but most of them shouldn't apply as the code has been removed.

I'll let this PR open if you want to add more changes on top of what we did, feel free to close it otherwise.

@ronanM ronanM closed this Apr 25, 2020
@ronanM ronanM deleted the init-gui branch April 25, 2020 07:15
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