rstudio / pins-r

Pin, discover, and share resources
https://pins.rstudio.com
Other
312 stars 63 forks source link

Having an argument "read_only" for "board" functions #684

Closed Sade154 closed 1 year ago

Sade154 commented 1 year ago

It would be nice to have an argument read_only to make sure that we dont alterate our data products when manipulating pins :

board_local(..., read_only = TRUE)

juliasilge commented 1 year ago

Thanks for the suggestion @Sade154! 🙌 We current turn off/on reading methods based off the class of the board, like so:

https://github.com/rstudio/pins-r/blob/95af5586bed14607caaea5610d438ad653793d7a/R/board_url.R#L134-L136

I think the kind of change you suggest would involve new classes of boards (like subclasses). Let's keep this issue open for understanding how important this feature could be and prioritizing it.

juliasilge commented 1 year ago

For now, we're going to pass on making a change in pins to have boards optionally read only. We can revisit this later if we get more feedback on it! 👍

github-actions[bot] commented 1 year ago

This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.