Skip to content
This repository has been archived by the owner on Nov 10, 2024. It is now read-only.

Raise error on unsupported parameters #470

Closed
llrs opened this issue Feb 9, 2021 · 1 comment
Closed

Raise error on unsupported parameters #470

llrs opened this issue Feb 9, 2021 · 1 comment

Comments

@llrs
Copy link
Collaborator

llrs commented Feb 9, 2021

From here it seems it would be better to alert users of unused arguments. Not sure if this could be made more general or a table/list would be nice (as a summary or cheatsheet):

library(rtweet)
search_tweets("hillaryclinton", n = 3, foo = "bar")

Works just fine even though foo is absolutely not a parameter supported by the Twitter API. They must ignore parameters they don’t support. This is same behavior as the GitHub API we saw above.

In the case of Twitter and GitHub one might want to raise errors on unsupported parameters client side in rtweet to avoid any use confusion of parameters being silently dropped.

@hadley
Copy link
Collaborator

hadley commented Feb 27, 2021

This is going to be very difficult to do, unless twitter provides a machine readable API description.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants