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

Leading hyphen "wl- " causes an error upon task entry #119

Closed
kaspersoerensen opened this issue May 3, 2016 · 5 comments
Closed

Leading hyphen "wl- " causes an error upon task entry #119

kaspersoerensen opened this issue May 3, 2016 · 5 comments
Assignees
Labels
Milestone

Comments

@kaspersoerensen
Copy link

Not really a bug - but maybe some error messaging could be nice.
Example: I punch in the following:
wl- div: Protocol due 11 may

Which gives this error:
skaermbillede 2016-05-03 kl 15 14 11

The error was that I typed wl- and then started typing the list name. That was a bit difficult to figure out by the error message.

Version of Alfred-Wunderlist-Workflow: 0.6.0-beta.3

@idpaterson
Copy link
Owner

@kaspersoerensen thanks for reporting, it looks like that hyphen indeed causes a problem when the task is added. The <?xml bit suggests that the workflow tried to handle your task entry as a normal query that you typed in to Alfred rather than as an action. When you press return the workflow includes a --commit at the end of the message to indicate that you want to perform the corresponding action, so I wonder if there is a problem parsing that --commit syntax when there is a preceding hyphen, as if it's stripping off your "- div: Protocol due 11 may" as another -- command...

Good find, I'll take a look at that this evening.

@idpaterson idpaterson changed the title Error messaging Leading hyphen "wl- " causes an error upon task entry May 3, 2016
idpaterson added a commit that referenced this issue May 4, 2016
@idpaterson idpaterson added this to the 0.6 milestone May 4, 2016
@idpaterson idpaterson added the bug label May 4, 2016
@idpaterson idpaterson self-assigned this May 4, 2016
@idpaterson
Copy link
Owner

idpaterson commented May 4, 2016

This was an easy fix, it will go out with the next release since it is not a high priority bug. Out of curiosity, did you arrive at wl- after using alt+delete to navigate back to the main menu from a different screen?

@kaspersoerensen
Copy link
Author

Hi,
Nice to hear! I actually just "misused" the command and typed wl- as I was first going to search for a task.

@idpaterson
Copy link
Owner

Thanks for following up, just a few more things to finish up before 0.6.0 is released so this will be in the next beta.

@idpaterson
Copy link
Owner

This is fixed in 0.6.0-beta.4

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

No branches or pull requests

2 participants