Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

KirshApps's First Goodie #1015

Merged
merged 6 commits into from Mar 4, 2015
Merged

KirshApps's First Goodie #1015

merged 6 commits into from Mar 4, 2015

Conversation

kirshapps
Copy link
Contributor

What does your Instant Answer do?

It tells people that I am awesome, who would otherwise not know.

What problem does your Instant Answer solve (Why is it better than organic links)?

It tells the world that I am awesome.

What are some example queries that trigger this Instant Answer?

Example: "duckduckhack kirshapps"

Which communities will this Instant Answer be especially useful for? (gamers, book lovers, etc)

This Instant Answer is good for everyone.

Where did you hear about DuckDuckHack? (For first time contributors)

By clicking a link from the DuckDuckGo Search Engine.

What does the Instant Answer look like? (Provide a screenshot for new or updated Instant Answers)

image

Checklist

Please place an 'X' where appropriate.

- [X] Added metadata and attribution information (https://duck.co/duckduckhack/metadata)
- [X] Wrote test file and added to t/ directory (https://duck.co/duckduckhack/goodie_tests)
- [X] Verified that instant answer adheres to design guidelines (https://duck.co/duckduckhack/design_styleguide)
- [X] Verified that instant answer adheres to code styleguide (https://duck.co/duckduckhack/code_styleguide)
- [X] Tested cross-browser compatibility

    Please let us know which browsers/devices you've tested on:
    - Windows 8
        - [X] Google Chrome
        - [X] Firefox
        - [] Opera
        - [] IE 10

    - Windows 7
        - [] Google Chrome
        - [] Firefox
        - [] Opera
        - [] IE 8
        - [] IE 9
        - [] IE 10

    - Windows XP
        - [] IE 7
        - [] IE 8

    - Mac OSX
        - [X] Google Chrome
        - [X] Firefox
        - [] Opera
        - [] Safari

    - iOS (iPhone)
        - [] Safari Mobile
        - [] Google Chrome
        - [] Opera

    - iOS (iPad)
        - [] Safari Mobile
        - [] Google Chrome
        - [] Opera

    - Android
        - [] Firefox
        - [] Native Browser
        - [] Google Chrome
        - [] Opera

topics "special_interest", "geek";
code_url "https://github.com/duckduckgo/zeroclickinfo-goodies/blob/master/lib/DDG/Goodie/IsAwesome/kirshapps.pm";
attribution github => ["https://github.com/kirshapps", "kirshapps"],
;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these two lines should be one line like so:

attribution github => ["https://github.com/kirshapps", "kirshapps"];

[qw(
DDG::Goodie::IsAwesome::kirshapps
)],
'duckduckhack kirshapps' => test_zci('kirshapps is awesome and has successfully completed the DuckDuckHack Goodie tutorial!'),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you decapitalize the g in goodie? This is what is making the tests fail.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Either makes sense...It's up to @kirshapps

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@javathunderman Hate to be that guy..... 😬 As "Goodie" is a proper noun it does start with a capital letter and is included in the tutorial this way, but overall it's a tiny change and either way would work.. 😄

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MrChrisW You're right, and don't worry...More than usual I am that guy. 😄

@kirshapps
Copy link
Contributor Author

Thanks @moollaza, I think I've got it now!

triggers start => "duckduckhack kirshapps";
handle remainder => sub {
return if $_;
return "kirshapps is awesome and has successfully completed the DuckDuckHack goodie tutorial!"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kirshapps Can you fix this hard-tab here? It should be 4 spaces 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mintsoft. How exactly would I do that?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kirshapps sorry for the slow reply; you could just delete the tab character and manually type 4 spaces; however the Codio editor should already be configured for spaces/"soft tabs":
image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mintsoft.

mintsoft added a commit that referenced this pull request Mar 4, 2015
@mintsoft mintsoft merged commit 01fb8ca into duckduckgo:master Mar 4, 2015
@mintsoft
Copy link
Collaborator

mintsoft commented Mar 4, 2015

Cool thanks @kirshapps Congrats on your first Goodie!

@moollaza
Copy link
Member

@kirshapps congrats! It's live: https://duckduckgo.com/?q=duckduckhack+kirshapps&ia=answer

Thanks again for taking the time to contribute! We really appreciate it. Feel free to stick around and comment/help on other PR's & Issues or even submit more Instant Answers!

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

Successfully merging this pull request may close these issues.

None yet

5 participants