manjaro Posted February 3, 2015 Share Posted February 3, 2015 Hello, Added a german verb conjugation workflow here Try it out, if you are eager enough to learn the intricacies of the german verbs and how it has to be conjugated! -Joe Link to comment
deanishe Posted February 3, 2015 Share Posted February 3, 2015 Hmm. The workflow appears to choke on non-ASCII queries, which is kind of a big thing when it's designed to accept German words… If I enter a query that contains an umlaut, the workflow sends XML to Alfred, but all the values are empty. Here's the result of entering the query "über": <?xml version="1.0" encoding="UTF-8"?> <items><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item><item valid="true" arg="" uid=""><title></title><subtitle></subtitle><icon></icon></item></items> Also, the escaping options for the Script Filter are wrong. If you use bash as the language you need to enclose {query} in double quotes ("{query}") and select the Backquotes, Double Quotes, Dollars and Backslashes escaping options. Escaping shouldn't be a problem with this workflow (I don't think verbformen.de has any multi-word entries), but "can't go wrong" > "should work". The GitHub repo isn't particularly fork-friendly. Most of the files required to build a working workflow (info.plist, icon.png) are only included in the .alfredworkflow file. Might I suggest you put all of the files required to build the workflow in the repo itself and the compiled .alfredworkflow file in a GitHub release? Or put the source files in a subdirectory if you want to keep the .alfredworkflow file in the repo. I'd advise against that, though, as GitHub won't let you put .alfredworkflow files >10MB in a repo (they have to go in releases). Finally, there's a workflows.php file in the workflow. Is this file actually required? Link to comment
manjaro Posted February 4, 2015 Author Share Posted February 4, 2015 Hello Deanishe, Thanks for the feedback! 1. über - showing empty XML entries => Was not handling the empty body. Now handled it. 2. umlauts not working => yep! were not working. Since I'm learning German, I missed out the most basic testing.. Thanks for pointing it out. Fixed it. 3. workflow contents in a separate folder => Done. 4. workflow itself in release => Done. 5. workflows.php => Yea, had earlier created an example workflow and missed to clean up. Done. Have updated the github repo. Please check and let me know. Have a great day! Joe. Link to comment
deanishe Posted February 4, 2015 Share Posted February 4, 2015 Klasse gemacht. Läuft wie geschmiert Link to comment
deanishe Posted February 4, 2015 Share Posted February 4, 2015 BTW, if you haven't done so already, you should probably post this in the Share Your Workflows forum. And perhaps on Packal, too. Link to comment
manjaro Posted February 5, 2015 Author Share Posted February 5, 2015 Hello Deanishe, Fixed another issue. Verbformen expects the URL of the verbs with umlauts as a: when the verb has ä and o:=> ö etc. Fixed and have done another minor release. You can take that. Also created a post in Share your Workflow and Packal http://www.packal.org/workflow/german-verb-conjugation Best Regards, Joe. Link to comment
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now