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

Bible: adding verse of the day #2936

Merged
merged 1 commit into from Aug 12, 2016

Conversation

gerlachry
Copy link
Contributor

@gerlachry gerlachry commented Aug 3, 2016

What does your Pull Request do (check all that apply)?

Choose the most relevant items and use the following title template to name
your Pull Request.

  • New Instant Answer: New {IA Name} Instant Answer
  • Improvement
    • Bug fix: {IA Name}: Fix {Issue number or one-line description}
    • Enhancement: {IA Name}: {Description of Improvements}
  • Non–Instant Answer
    • Other (Role, Template, Test, Documentation, etc.): {SpiceRole/Templates/Tests/Docs}: {Short Description}
Description of changes

Adding

bible verse of the day

triggers and tests. Uses the NET Bible API verse of the day. The default 1 day caching seems ok for this verse of the day or should this one also be split out to a separate IA like what is being suggested in #2826 for random bible verse and then enable a much longer cache duration on this current IA?

Which issues (if any) does this fix?

Fixes #2929 - Adds regex for verse of day and various tests.

People to notify (@mention interested parties)

@pnpninja @tagawa @hunterlang


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

Maintainer: @mention

@daxtheduck
Copy link

@gerlachry Hey!

Thanks for taking the time to contribute! We really appreciate it.

We work closely with every contributor to make Instant Answers the best they can be, so we appreciate your patience as we look over your code. From here, the process usually goes like this:

  1. Pull Request is reviewed by the DuckDuckGo staff and community
  2. Staff and community will leave feedback with any necessary updates to the function or design.
  3. Once you've made any necessary corrections, then your Instant Answer will be merged and deployed live on DuckDuckGo!

If you have any questions along the way, feel free to ask them here. Our staff and community are also available on Slack to answer any questions you may have. If you'd like to join us there please head to https://quackslack.herokuapp.com/ to get an invite.

More Info: http://docs.duckduckhack.com/submitting/submitting-overview.html

Thanks!

@daxtheduck
Copy link

daxtheduck commented Aug 3, 2016

Bible

Description: Provides the text of selected Bible verses from the NET Bible translation.

Example Query: [genesis 15:7](https://beta.duckduckgo.com/?q=genesis 15:7), [bible genesis 26:4](https://beta.duckduckgo.com/?q=bible genesis 26:4)

Tab Name: Answer

Source:

These are the important fields from the IA page. Please check these for errors or missing information and update the IA page


This is an automated message which will be updated as changes are made to the IA page

@tagawa
Copy link
Collaborator

tagawa commented Aug 5, 2016

@gerlachry Thank you for adding this. I think a day's caching would be fine and combining it as you've done here will work. However there seems to be an issue installing it on the beta server. Not sure what's going on so I'll try to look into it.

@gerlachry
Copy link
Contributor Author

@tagawa Thanks. If need something from me or additional updates for deploying let me know.

@tagawa
Copy link
Collaborator

tagawa commented Aug 8, 2016

Hi @gerlachry. I've looked into this a bit more but can't see the problem. Your code looks fine, it works for me in Codio, and all the tests pass. I've asked for some expert help from a colleague so hopefully we can get it fixed soon.

By the way, when it works, here's the link that should show your update: https://beta.duckduckgo.com/?q=bible+verse+of+the+day&ia=answer

@tagawa
Copy link
Collaborator

tagawa commented Aug 8, 2016

Thanks to @jdorweiler working his magic, this is now working on the beta server for testing: https://beta.duckduckgo.com/?q=bible+verse+of+the+day&ia=answer
Let's give it a few days to see if there are any review comments, and after that we can merge.

@tagawa
Copy link
Collaborator

tagawa commented Aug 12, 2016

@gerlachry We've had no further review comments and this is working well for me on the beta server (and the previous functionality is unaffected) so let's merge! Thanks again for this improvement 😄

@tagawa tagawa merged commit cd95500 into duckduckgo:master Aug 12, 2016
@gerlachry gerlachry deleted the gerlachry/bible-verse-of-day2 branch August 12, 2016 11:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bible verse of the day
4 participants