Skip to content

Adding new countries and languages that were missing#41

Merged
mattlisiv merged 1 commit intomattlisiv:masterfrom
menarguez:newsapy-python-new-countries-and-languages
Sep 6, 2019
Merged

Adding new countries and languages that were missing#41
mattlisiv merged 1 commit intomattlisiv:masterfrom
menarguez:newsapy-python-new-countries-and-languages

Conversation

@menarguez
Copy link
Copy Markdown
Contributor

  • New countries: ["pk", "es", "is", "se", "zh"]
  • New languages:["se",
    "zh"])

 - New countries: ["pk", "es", "is", "se", "zh"]
 - New languages:["se",
"zh"])
@mattlisiv
Copy link
Copy Markdown
Owner

mattlisiv commented Jul 30, 2019

@menarguez I do not see these new additions in the official documentation online. Do you have a source where you found these?

Thanks!

@menarguez
Copy link
Copy Markdown
Contributor Author

I did so by pulling all sources from the API. As I was using the const as the True value list of candidate items, storage of sources failed. Adding these fixed the problem

@bsolomon1124
Copy link
Copy Markdown
Collaborator

These are referenced here:

https://newsapi.org/docs/endpoints/everything

(Looks like i accidentally duplicated part of this PR in #44. Sorry for not looking here first.)

@mattlisiv
Copy link
Copy Markdown
Owner

Great. At the time, the official documentation did not have the new additions, so I was hesitant to add, but all good now.

Thanks!

@mattlisiv mattlisiv merged commit 21ed2b8 into mattlisiv:master Sep 6, 2019
@menarguez
Copy link
Copy Markdown
Contributor Author

It has been a while but thanks for merging this back in the day and thanks for the new additions as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants