LearnToDiscover / sandpaper

L2D's fork of {sandpaper}
https://learntodiscover.github.io/sandpaper/
Other
0 stars 1 forks source link

Add better error reporting for missing `config` variables #85

Open milanmlft opened 5 months ago

milanmlft commented 5 months ago

When the ipynb or pdf variables are missing from a lesson's config.yaml, building a lesson results in an error that is not very clear about what is going wrong:

Error in if (cfg$ipynb) { : argument is of length zero

which is the error you get when passing NULL in if().

We could make this error message clearer to notify the user about the missing value. Alternatively, we could make it so that when the value is missing, we can treat it as being equivalent to FALSE.