Zeroclickinfo-goodies: Timer: Improve triggering

Created on 24 Mar 2017  路  7Comments  路  Source: duckduckgo/zeroclickinfo-goodies

Problem

Based on user feedback and unanswered query data, This IA isn't triggering enough. @moollaza just merged a PR that ported this IA from a Spice to a Goodie.

Triggering 馃憤

screen shot 2017-03-24 at 6 55 21 pm

Not triggering 馃憥

screen shot 2017-03-24 at 6 55 38 pm

Solution

  1. Add one, two, or three tests! 馃槈
  2. Add the trigger words mins, minutes

People to notify


@pjhampton

Get Started

Resources


Instant Answer Page: https://duck.co/ia/view/timer

Highest Impact Tasks Low Improvement Low-Hanging Fruit Work In Progress Other

All 7 comments

Is anyone working on this? I would like to, if it's possible. (I'm new btw, I'm still reading the contrib guide 馃槤)

@TheSpeedyBlur Great! Sure, go ahead with this. If you face any issue, feel free to ping on Slack.

Okay, I'm sorry, I was too dumb to look at this issue in detail, and I would like to cancel my application for this issue. I'm new to the programming language (perl) and this task is too complex for me at the first try. (I should probably should look up some JS issues, cause I'm better at that) I've been sitting on it for two hours, and again, I'm sorry for dropping this, but I'm afraid I would only be able to accomplish progress if I would rewrite the whole Timer script. 馃槥

@TheSpeedyBlur I know perl can be a bit intimidating (I'm relatively new to it myself), but I don't think this would be too hard with a bit of help. I'd be willing to point you in the right direction if you still want to take the issue. Ping me on Slack (my username there is the same as my Github) and let me know. We could figure out this one together.

@BillyBarbaro I'd be happy to do this actually and if you're willing to help me I'd be glad. I will ping you on slack when I'm available.

Thanks very much @TheSpeedyBlur and @BillyBarbaro!! I noticed you're not part of the DDG org on Github @TheSpeedyBlur, so I have only assigned @BillyBarbaro for the time being. 馃挋 馃挍 馃挏 鉂わ笍

@pjhampton Should we trigger on hours / hrs, seconds / secs as well?

Also, what should I do about this test?
https://github.com/TheSpeedyBlur/zeroclickinfo-goodies/blob/c35ed3e315798cf5c25aec0fa38653d180dce3df/t/Timer.t#L103

It keeps failing every time, but me and @BillyBarbaro think that it could match. Can I remove it?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

moollaza picture moollaza  路  5Comments

zappe picture zappe  路  5Comments

jc86035 picture jc86035  路  4Comments

germangb picture germangb  路  5Comments

fzzr- picture fzzr-  路  5Comments