search

UBC-MDS / ds-toolbox

https://toolbox-learn.mds.ubc.ca/
Other
3 stars 6 forks source link

Revise module 6 #77

Closed joelostblom closed 2 years ago

netlify[bot] commented 2 years ago

Deploy Preview for cheerful-chebakia-6a18e7 ready!

Name Link
Latest commit 490778ff2aa7608f1f47f146474ebe4f32d951ef
Latest deploy log https://app.netlify.com/sites/cheerful-chebakia-6a18e7/deploys/62f0ac33228535000836db1c
Deploy Preview https://deploy-preview-77--cheerful-chebakia-6a18e7.netlify.app/en/module6
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] commented 2 years ago

Deploy Preview for ds-toolbox ready!

Name Link
Latest commit 490778ff2aa7608f1f47f146474ebe4f32d951ef
Latest deploy log https://app.netlify.com/sites/ds-toolbox/deploys/62f0ac33d6d667000aa7635f
Deploy Preview https://deploy-preview-77--ds-toolbox.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

armanawn commented 2 years ago

@joelostblom This looks great to me, thank you for the revision! Just wondering, are we going to replace the current practice question titles to "✍️ Practice..."? I see that this has been done to some of the modules but not to this one.

joelostblom commented 2 years ago

Thanks for the comments and quick fixes @armanawn ! I have added the "Practice" prefix to the exercises in my last commit. If this all looks good to you then maybe either one of us can go ahead and squash and merge the PR? And you can of course always make more fixes later as you do your recording of these slides.