Skip to content
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

Bugfix/issue 108 #109

Merged
merged 2 commits into from
Oct 14, 2020
Merged

Conversation

CannoliSquid
Copy link
Contributor

Added the additional parameters supported by ISteamNews/GetNewsForApp. Made sure to utilize the SteamNewsTests unit test to make sure it didn't break as well as ran the calls with additional parameters (which I left in, uncommented) to make sure that they reduced the number of results and functioned as expected.

Feeds is a string and Tags is a string array. The changes passed unit tests.
Shows that the parameters actually filter down the results.
@babelshift babelshift merged commit 7f194c1 into babelshift:master Oct 14, 2020
Trekkan pushed a commit to Trekkan/SteamWebAPI2 that referenced this pull request Mar 22, 2021
* Accommodate Changes to Add "Feeds" and "Tags" as SteamNews Parameters

Feeds is a string and Tags is a string array. The changes passed unit tests.

* Adding Parameter Examples

Shows that the parameters actually filter down the results.
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.

2 participants