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

Allow user to choose how to run commands #13

Open
sayalirkakade opened this issue Nov 5, 2016 · 4 comments
Open

Allow user to choose how to run commands #13

sayalirkakade opened this issue Nov 5, 2016 · 4 comments

Comments

@sayalirkakade
Copy link
Contributor

Instead of hard coding the commands, let the user choose what text will trigger which action so that the program is more flexible. Additionally, this could fix potential bugs where the user writes the command wrong. For example, when sending a message through the SMS command, the user must enter m: and a space after the [number] and before the [message]. However, if the user forgets the "m: ", their message is truncated and the start of the message is not send. There is also no error handling there to ensure that the user knows this might happen. Allowing the user to choose how to run commands would enable them to set the command names to what they want.

@jimmylle
Copy link
Contributor

jimmylle commented Nov 9, 2016

@sayalirkakade Can you explain how the user would choose what text will trigger what action? Will there be like an input where they can put the string that will trigger the action. What about the commands that take in an argument like "//Contact [name]"?

@sayalirkakade
Copy link
Contributor Author

Yes, once the user creates an instance of the application, there will be a place in the settings where they can change the text that triggers the certain action. After saving it, whenever they send that text while the application is still running, the command will execute. This is more of an enhancement type idea for customization. To implement it, you would probably need a dynamic variable. With the commands that take in an argument, the user can't change the format of the argument, (for example, SMS will still require a 10 digit number followed by a message), but they can change the actual command from "SMS" to whatever they prefer.

@jimmylle
Copy link
Contributor

F16 OK (200 pts)

@seemantasaha
Copy link
Contributor

seemantasaha commented Jan 23, 2018

W18 OK. 200 points.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants