-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
get_news() does not take into account the date range #99
Comments
Time period is not always working with Google, sometimes Google will return data out of specific range. Anyway, start/end filter might be able to add, will update it in next release. |
I came across the same issue, PashaM999's solution seems to work. |
Nice solution! Any plan to fix it? |
It's from Google, we can't do anything about it. |
Em ... I have tried to add |
I am trying this on 1.6.12 version. after my code (shown below) was only returning the recent news.
I am editing now init.py (seems the self.url line is now on 273) |
Is it possible to include @PashaM999 in future updates? That would be helpful |
Hi, I have been looking through the code and found that the function
get_news
has the url generated as follows:(line 259 in
__init__.py
)This uses the
self.__period
variable, which is only responsible for periods like 7d 1m and etc. Google has a search filter to use for specific dates, which can be implemented into your code as follows:This is merely a suggestion, but I feel like if the
__start
and__end
variables are set, you sould prioritize this over your original solution.Hope that will be useful for someone :)
The text was updated successfully, but these errors were encountered: