lhCore::checkin_lessons(): Checking in lessons from course 2026_delta…#11
lhCore::checkin_lessons(): Checking in lessons from course 2026_delta…#11
Conversation
…_week1 to branch 2026_delta_week1
…_week1 to branch 2026_delta_week1
…_week1 to branch 2026_delta_week1
|
@njlyon0 @regetz hey, please take a look at this pull request for the Lego activity. Let me know if it looks good to go to check it in to the lhLessons repository/package as the "canonical" Lego activity (for now). For future iterations of the lhCore checkin_lessons() might be cool to at least include the name of the files being checked in, with some explanatory text aside from the boilerplate I have above... |
|
Hey Casey! Broadly, this is looking good to me! That said, it feels like a lot of ancillary files to commit to the repo which could eventually cause the repo to bloat with these sorts of additional PDF resources. I'm not sure what the generic solution is but at least for the LEGO activity I don't know if we need any/most of those PDFs. Sure, yes, the PDF instructions are potentially useful but in order to teach the lesson (as it appears in the relevant qmd file) you only need the Quarto file and the links embedded therein. I know it's not necessarily a problem now but I could see a world where people throw in a number of bonus PDFs for every lesson and this package winds up being a huge volume of data for someone to install on their computer when most of that data is static PDFs that instructors don't necessarily need (however useful the context they provide is). Does that make sense? Happy to chat more if that's useful! |
|
Good point; a few ideas:
Most lessons won't need handouts (i.e., printable and/or form-fillable separate documents), so I don't see this being a major problem going forward, but worth considering as long as we're thinking about it! For now, as a compromise I will delete the .pdfs and keep only the .docx version for the handouts, and include the instructions directly in the lesson. (This will just be for the lhLessons; I'll leave it as is for Delta week 1) |
…_week1 to branch 2026_delta_week1