search

carpentries / instructor-training

Instructor Training
https://carpentries.github.io/instructor-training/
Other
175 stars 290 forks source link

Update instructor-notes.md #1573

Closed karenword closed 8 months ago

karenword commented 1 year ago

Cleaning up out-of-date info.

(Not actually anythig related to the checkout project here, just lots of other stuff to update instead, hence separate PR that can be merged anytime).

github-actions[bot] commented 1 year ago

:ok: Pre-flight checks passed :smiley:

This pull request has been checked and contains no modified workflow files, spoofing, or invalid commits.

Results of any additional workflows will appear here when they are done.

ndporter commented 1 year ago

Looks great and helpful overall: A few notes on specific lines below:

Line 39 "ad-blocking strategy" - this is relevant to both workshop types. Also "ad management" is probably better (with ideal being use an ad-free account or prewatch the ads day-of and cue up the video after them, since ad blockers are somewhere in a murky space, legal but maybe not the best to recommend officially?).

Line 52 - "remind trainees to fill out the application" - what application is this that they need reminded during the event? Whatever it is, there should probably be a link and/or explanation here.

Line 58 - why did this change from required to optional and remove the checkout linking? Is that function now being managed through other systems like AMY?

Line 160 - should we perhaps leave at least 1 link to a general guide on Zoom hosting if we remove all the detailed instructions? There may be people who are not used to Zoom or to hosting on Zoom and aren't aware what it can do, especially outside academia or in the global south.

Line 170ff - the conventions for initials (FL or F.L. or F.L or FL.) are inconsistent here.