Skip to content
This repository has been archived by the owner on Jun 29, 2018. It is now read-only.

Wikipedia fix #465

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Wikipedia fix #465

wants to merge 2 commits into from

Conversation

staplegun
Copy link

This is a fix for #464 so the correct parameters are used in the Wikipedia API call.

I also changed the API call so it only returns the main summary. Often long Wikipedia entries contain table of contents, etc. that look odd in a single block of text. The API 'extract' call lets us just get the main summary at the top of the Wikipedia entry. This means it will be shorter, but probably more meaningful.

@NathanLawrence
Copy link

This seems like a really good way to handle this situation -- things are definitely very broken right now.

@kirkins
Copy link

kirkins commented Jan 30, 2017

I take it Mozilla never looks at pull requests for this repo?

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.

3 participants