-
-
Notifications
You must be signed in to change notification settings - Fork 276
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 a parameter to hide the waiting message in the GUI #1141
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, a few changes to be made though.
Also please be sure to add yourself to the AUTHORS
file, and update the documentation with the new parameter.
Please also fix your commit message to follow the standard this project uses, for example:
[client] app: add option to disable the waiting for host message
1ded9f3
to
354be42
Compare
354be42
to
79264a4
Compare
It should be all good to go, now. Let me know if you need additional changes merged here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing the requested changes. Documentation edits were just updating the client-side params, and this appears to have been done correctly.
Just a small change to hide the popup messagebox in the UI while waiting.