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

Confirm custom command Req #532 #590

Merged
merged 2 commits into from Sep 23, 2014
Merged

Conversation

@Booli
Copy link
Contributor

Booli commented Sep 23, 2014

Added confirmation option to custom controls. Same syntax (and code) as system commands confirmation.

"confirm": "Would you really like to do custom command?"

First commit, after a struggle got it to devel branch.

Booli added 2 commits Sep 23, 2014
Syntax (and code) similar to System Command confirm. First commit,
please be gentle.
@Booli

This comment has been minimized.

Copy link
Contributor Author

Booli commented Sep 23, 2014

Hope I did this correct!

@foosel foosel merged commit 50eac58 into foosel:devel Sep 23, 2014
@foosel

This comment has been minimized.

Copy link
Owner

foosel commented Sep 23, 2014

Yep, perfect, thanks :)

@Booli

This comment has been minimized.

Copy link
Contributor Author

Booli commented Sep 23, 2014

Cool! :)

@Booli Booli deleted the Booli:confirmCustomCommand branch Sep 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.