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

Port non-API Spice to Goodies #2463

Closed
5 of 6 tasks
GuiltyDolphin opened this issue Feb 1, 2016 · 20 comments · Fixed by #2910
Closed
5 of 6 tasks

Port non-API Spice to Goodies #2463

GuiltyDolphin opened this issue Feb 1, 2016 · 20 comments · Fixed by #2910

Comments

@GuiltyDolphin
Copy link
Member

GuiltyDolphin commented Feb 1, 2016

Goodies can support JS too! Any Spice that doesn't require API calls should be ported to Goodies, if you're interested, please follow the Guidelines below.

Guidelines

Choosing a Spice

  1. Pick an available Spice from the Relevant Spices list (if you have mentioned that you would like to port a Spice, but can no longer do so for any reason, please let me know so I can make it clearly available again).
  2. Comment to let me know which Spice you are interested in porting, feel free to inquire about Spices that are taken but haven't seen any development for a long time.

Porting the Spice

  1. If you want to move any of the JS into Perl, feel free, but the functionality must remain the same (_if you are extending the functionality at all, please make that clear in your pull request!_).
  2. Update the tests accordingly - the test format will no longer be the same. If you are able to do so, please update the tests to actually show results rather than just responses.
  3. Make sure the tests pass (use duckpan test to test everything and duckpan test SPICE_NAME (where SPICE_NAME is the name of the Spice being ported) to test the Spice).

Uploading your changes

  1. Make a pull request with the title Port SPICE_NAME to Goodies where SPICE_NAME is the name of the Spice you are porting. You should make one PR on Spice, which removes the Spice, and one on Goodies, which ports the functionality. _Please link to the PR on the other repo as well as this issue for both pull requests!_
  2. Add a link to the IA page of the Spice in the PR description. Do not create a new IA page for the Goodie version.
  3. If you have changed the functionality at all, please let us know in the pull request!

Relevant Spices

  • Editor
  • Timer
  • Geometry
  • ColorPicker
  • Stopwatch
  • FlashVersion

Reference to the Goodie issue: duckduckgo/zeroclickinfo-goodies#2258

@GuiltyDolphin
Copy link
Member Author

If you spot a Spice that should (or shouldn't) be on this list, please let me know!

@jagtalon
Copy link
Member

jagtalon commented Feb 8, 2016

@GuiltyDolphin this is really useful! ❤️ ❤️ ❤️

@GuiltyDolphin
Copy link
Member Author

@jagtalon I'll kick this off by porting Timer as an example.

@jagtalon
Copy link
Member

jagtalon commented Feb 8, 2016

@GuiltyDolphin yeaaaaaa

@theaverageguy
Copy link

I would like to take up FlashVersion. :)

@GuiltyDolphin
Copy link
Member Author

@yash14123 Sure thing!

Remember to reference this issue in both PR's (and a reference to the other PR in each) when you're ready! :)

@moollaza
Copy link
Member

moollaza commented Apr 5, 2016

@duckduckgo/duckduckhack-contributors we could really use a lot of help over here too. This is a fairly easy, but very important task.

We didn't always have JS available to Goodies and so we have a few "Spice" Instant Answers that don't actually use APIs and they should really be converted to Goodies with JS (like the 2048 Game).

There are still 4 IA's left to convert!

Is anyone interested in tackling them?

@moollaza
Copy link
Member

moollaza commented Apr 5, 2016

@theaverageguy are you still interested in converting FlashVersion? If so don't forget to mention this issue in your PR so everyone else knows it's been fixed!

@mattr555
Copy link
Collaborator

mattr555 commented Apr 7, 2016

I'll port the stopwatch and color picker, as those are my other JS-only spices (I tend to make a bunch of those)

@zekiel
Copy link
Member

zekiel commented Apr 8, 2016

You are my hero @mattr555

@shashankgroovy
Copy link
Collaborator

hey @mattr555 ! Are you still up for porting the color picker spice? If not I'd like to do it as my first contribution. And @GuiltyDolphin issue #2675 is pending for color picker - awaiting a better solution . so should we wait for it or can I fix that issue and port color picker together?

@GuiltyDolphin
Copy link
Member Author

@shashankgroovy I think that solution won't be necessary as we'll be processing more of the queries in the Perl so I think it would be better handled in Perl as a Goodie (though I haven't familiarized myself with the workings of that IA).

What say you @mattr555 ?

@shashankgroovy
Copy link
Collaborator

@GuiltyDolphin will the geometry and editor spices will also be rewritten in Perl as a goodie as well?
If not then I was hoping maybe I could pick up either one of them

@GuiltyDolphin
Copy link
Member Author

@shashankgroovy Geometry has a PR in progress (duckduckgo/zeroclickinfo-goodies#2675); but I don't think there's anything in progress for Editor yet - you can pick it up if you wish ❤️

@shashankgroovy
Copy link
Collaborator

shashankgroovy commented Jun 10, 2016

@GuiltyDolphin That'd be great. I'll take up the Editor. I will start working on it asap.
I hope I can ask you for some guidance over email or slack as its it'd be my first contribution. Thanks!

@edgesince84
Copy link
Collaborator

@shashankgroovy awesome!

@GuiltyDolphin
Copy link
Member Author

@shashankgroovy Sure, feel free to ping me on Slack if you have any issues.

@manrajgrover
Copy link
Contributor

@GuiltyDolphin @edgesince84 @moollaza I can start porting Editor if @shashankgroovy is okay with it. 😄 Also, if there are more IAs to be ported, please mention them.

@manrajgrover manrajgrover mentioned this issue Jul 20, 2016
6 tasks
@shashankgroovy
Copy link
Collaborator

@manrajgrover yep go ahead!

@GuiltyDolphin
Copy link
Member Author

@manrajgrover I haven't spotted any - though if you see one please comment here to let me know and I can add it to the list ❤️

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

Successfully merging a pull request may close this issue.

9 participants