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.

Sign up for GitHub

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

Jump to bottom

Including white spaces into /todo send crashes the plugin #70

Closed
hanzei opened this issue May 28, 2020 · 1 comment · Fixed by #86
Closed

Including white spaces into /todo send crashes the plugin #70

hanzei opened this issue May 28, 2020 · 1 comment · Fixed by #86
Assignees
@larkox
Labels
Tech/Go Type/Bug Something isn't working

Comments

@hanzei
Copy link
Contributor

hanzei commented May 28, 2020

e.g. /todo send admin
@hanzei hanzei added Help Wanted Community help wanted Tech/Go Up For Grabs Ready for help from the community. Removed when someone volunteers Type/Bug Something isn't working labels May 28, 2020
@mickmister
Copy link
Contributor

This could affect other plugins with similar arg parsing logic

This was referenced Jun 19, 2020
Closed
Merged
@larkox larkox linked a pull request Jun 19, 2020 that will close this issue
Allow extra spaces between commands #86
Merged
@larkox larkox removed Help Wanted Community help wanted Up For Grabs Ready for help from the community. Removed when someone volunteers labels Jun 23, 2020
@larkox larkox self-assigned this Jun 23, 2020
@larkox larkox closed this as completed in #86 Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@larkox larkox

Labels
Tech/Go Type/Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow extra spaces between commands larkox/mattermost-plugin-todo
3 participants
@larkox @mickmister @hanzei