Skip to content

Conversation

@OlegRa
Copy link

@OlegRa OlegRa commented Apr 26, 2018

EverythingRequest fields applied incorrectly inside GetEverythingAsync method. Sources list used for populating domains list. This problem affects both sources- and domain-based filters.

EverythingRequest fields applied incorrectly inside GetEverythingAsync
method. Sources list used for populating domains list. This problem
affect both sources- and domain-based filters.
@KarmaCop213
Copy link

After almost 1 year, no fix yet.

@OlegRa
Copy link
Author

OlegRa commented Feb 12, 2019

After almost 1 year, no fix yet.

You can build binaries with fix from my fork or push News API guys to accept this PR if you have subscription. I did this fix for one of my client and have no mood for forcing this PR acceptance.

Copy link

@joviledog joviledog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need to get the domains fixed too.

dependabot bot and others added 2 commits June 23, 2022 01:01
Bumps [Newtonsoft.Json](https://github.com/JamesNK/Newtonsoft.Json) from 10.0.3 to 13.0.1.
- [Release notes](https://github.com/JamesNK/Newtonsoft.Json/releases)
- [Commits](JamesNK/Newtonsoft.Json@10.0.3...13.0.1)

---
updated-dependencies:
- dependency-name: Newtonsoft.Json
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
….Json-13.0.1

Bump Newtonsoft.Json from 10.0.3 to 13.0.1 in /NewsAPI
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