You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First off, thanks for creating and releasing this very helpful Package, it saved me a lot of time from coding it on my own for my quick project.
The only suggestion I have is in reference to the formatting of the return value for 'description'
What gets returned to me is not a description of the article but a series of short titles from other news sites, without links.
I know you are running it through BeautifulSoup which removes the links and the list structure and what is left is a confusing mess.
I modified the code so that I get back everything I want, but for other users you may want to add an option to switch that on and off. I added this to my base and control it during initialization of GNews, now I switch formatting by BeautifulSoup on/off with a simple option I pass once anytime it's needed. Considering most consumers of your API are technical this will not be confusing.
The text was updated successfully, but these errors were encountered:
First off, thanks for creating and releasing this very helpful Package, it saved me a lot of time from coding it on my own for my quick project.
The only suggestion I have is in reference to the formatting of the return value for 'description'
What gets returned to me is not a description of the article but a series of short titles from other news sites, without links.
I know you are running it through BeautifulSoup which removes the links and the list structure and what is left is a confusing mess.
I modified the code so that I get back everything I want, but for other users you may want to add an option to switch that on and off. I added this to my base and control it during initialization of GNews, now I switch formatting by BeautifulSoup on/off with a simple option I pass once anytime it's needed. Considering most consumers of your API are technical this will not be confusing.
The text was updated successfully, but these errors were encountered: