-
Notifications
You must be signed in to change notification settings - Fork 7
Error in #2
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
Comments
Apologies for the delay in responding. Are you using the CRAN version? |
Ahh. I think you need to include library(tutorial.helpers) in your tutorial Rmd file, just as you need to include library(learnr). |
Hi David, and thanks for your fedback. It might be something else, since both packages are included in the first chunks of that Rmd file (I cloned one of the examples, trying to tweak just a few things just as some sort of hello-world approach) I exposed my simple tweak in a Rmd file here: |
You're right! I can replicate this error. Will fix now. |
Should be fixed. Can you please:
And try again? |
Awesome, that was a fast fix! The three buttons became functional after this last commit.
(which is even shorter, btw, plus allowing better reproducibility across machines/os/environments, etc) |
Nice! By the way, you are my first user. I am eager for any feedback/requests you might have! dave.kane at gmail |
Great, thanks David. Btw, I added a request already, to the other package :-) : How to contribute translations to these nice tutorials you are doing Best |
Hi, and thanks for creating this package.
I need to create some course content, and I'm reviewing your package to see whether I want to follow the route of the learnr package, or something different (since content will end up in a Moodle environment).
When clicking at "Run Document" on the first hello-world tutorial I'm tweaking, I get, errors like:
every time I click at the buttons to download the answers (any of them: "Download RDS", "Download HTML" or "Download PDF"). the dialogue to download the file is shown, but no file is saved on disk indeed.
Hope this helps
The text was updated successfully, but these errors were encountered: