Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

parse param in core_articles_pdf() doesn't make a whole lot of sense #11

Closed
sckott opened this issue Nov 24, 2017 · 0 comments
Closed
Milestone

Comments

@sckott
Copy link
Contributor

sckott commented Nov 24, 2017

given that core_articles_pdf_ exists and is exported to users. that is, core_articles_pdf_ just downloads and gives back the file path, AND that's what core_articles_pdf(id, parse = FALSE) does too

therefore, should drop parse parameter in core_articles_pdf

@sckott sckott added this to the v0.3 milestone Nov 24, 2017
@sckott sckott closed this as completed in 052cc39 Nov 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant