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

Add limit argument to get_status() function #102

Closed
wants to merge 2 commits into from
Closed

Add limit argument to get_status() function #102

wants to merge 2 commits into from

Conversation

smach
Copy link

@smach smach commented Nov 21, 2022

Suggesting a very tiny change to add argument limit to get_status() function in order to use Mastodon limit parameter allowing return of up to 40 items instead of the default 20. Thank you for this package!

@chainsawriot
Copy link
Collaborator

chainsawriot commented Nov 21, 2022

@smach Thanks for the suggestion. But we already have this function for getting statuses of a user

https://github.com/schochastics/rtoot/blob/f9548a1f441d5b522898093779e2c17402e1fe84/R/accounts.R#L54-L61

Whereas get_status() is for querying one status.

@smach
Copy link
Author

smach commented Nov 21, 2022

OK sorry, I was confused, I still didn't see any way to change the default limit on get_account_statuses() either, but not important. I can look tonight after work and see if it's worth adding to that or I should just close this. Thanks again.

@schochastics
Copy link
Member

schochastics commented Nov 21, 2022

The function get_status connects to this endpoint which, as mentioned, only retrieves one status. get_account_statuses() access this endpoint which indeed has several parameters which i overlooked!

Edit: I opened an issue (#103) I will fix it this afternoon.

@smach
Copy link
Author

smach commented Nov 22, 2022

I see you already made an update, thank you!

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