-
Notifications
You must be signed in to change notification settings - Fork 9
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 text argument #64
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few minor requests, then it's good to go. Thanks @LewisAJones!
Hi @willgearty, I've implemented your suggestions! The |
Whoops, you forgot to move knitr from suggests to imports @LewisAJones! |
Fixed! I'll merge once the checks pass, thanks! |
This PR adds the
text
argument toget_attribution
. This allows the user to return a text paragraph of the attribution information to insert directly into a figure caption or similar.I am not particularly happy with the approach for getting the license information. I originally thought I could just pull this from the PhyloPic API but when testing this, it doesn’t seem to return all the different licenses. Therefore, this is hardcoded for now, but could be updated!
These updates also now allow
get_attribution
to be called with multiple uuids at once.Closes #56.