mattlisiv / newsapi-python

A Python Client for News API
MIT License
376 stars 130 forks source link

Adding new countries and languages that were missing #41

Closed menarguez closed 4 years ago

menarguez commented 4 years ago
mattlisiv commented 4 years ago

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

Thanks!

menarguez commented 4 years ago

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 commented 4 years ago

These are referenced here:

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

(Looks like i accidentally duplicated part of this PR in https://github.com/mattlisiv/newsapi-python/pull/44. Sorry for not looking here first.)

mattlisiv commented 4 years ago

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

Thanks!