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

Fix save_reactable error in R4.3.1 #59

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dcomputing
Copy link

R/save_reactable.R: fixed to match R4.3.1 handling of &&

R/save_reactable.R: fixed to match R4.3.1 handling of &&
@dcomputing dcomputing changed the title DESCRIPTION: version bumped to 2.1.1 Fix save_reactable error in R4.3.1 Aug 1, 2023
@cararthompson
Copy link

cararthompson commented Nov 8, 2024

Thank you for sharing your solution! I have pinched it (with credit!) for use in a bespoke package in which I have a function that creates a reactable that the client will need to export. Looking forward to seeing the fix in the CRAN version hopefully soon.

@Svyk
Copy link

Svyk commented Nov 30, 2024

Thank you for sharing your solution! I have pinched it (with credit!) for use in a bespoke package in which I have a function that creates a reactable that the client will need to export. Looking forward to seeing the fix in the CRAN version hopefully soon.

how would I update my R environment to get update my package with this solution?

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