Skip to content
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

Update instructor-notes.md #1573

Merged
merged 4 commits into from
Feb 28, 2024
Merged

Update instructor-notes.md #1573

merged 4 commits into from
Feb 28, 2024

Conversation

karenword
Copy link
Contributor

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).

Cleaning up out-of-date info
@github-actions
Copy link

github-actions bot commented Jul 25, 2023

🆗 Pre-flight checks passed 😃

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.

github-actions bot pushed a commit that referenced this pull request Jul 25, 2023
@ndporter
Copy link
Contributor

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.

github-actions bot pushed a commit that referenced this pull request Dec 28, 2023
instructors/instructor-notes.md Outdated Show resolved Hide resolved
- If teaching remotely: Ensure that you are logged into the Zoom room with the host key so that you can mute participants and create breakout rooms as needed.
- If teaching remotely: Turn off video during long exercises and breaks and disconnect during lunch (note: you may need to re-log in with the host key if returning after disconnecting).
- For online events, ensure that you are logged into the Zoom room with the host key so that you can mute participants and create breakout rooms as needed.
- For self-organised training events, it may be necessary to remind trainees to fill out the application.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- For self-organised training events, it may be necessary to remind trainees to fill out the application.
- For self-organised training events, it may be necessary to remind trainees to fill out the [application](https://amy.carpentries.org/forms/request_training/). Be sure to have trainees select "Profile Creation for Pre-approved Trainees" and enter their group code; contact [email protected] if you are not sure what code to use.

- Send a list of those who completed the training to checkout@carpentries.
- Send an email to trainees thanking them for participating and linking to [checkout checklist][checkout-checklist]. See [template][post-template].
- Double check that your attendance sheet is complete. For self-organized events, email instructor.training@carpentries.org to be sure we have your attendance record.
- If you wish, you can an email to trainees thanking them for participating.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ndporter the checkout checklist that was linked here is very outdated. At this time we feel that our pre-training email and follow-up reminder (sent ~2 months after training) are sufficient in providing all the necessary information that trainees will need.

@@ -153,30 +159,15 @@ you will be asked for a 6-digit host key, which you can find [in this message](h
See [the explanation here][zoom-host-key] for step-by-step instructions on how to
claim host in Zoom Rooms using the host key.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ndporter if you'd like to add a link to the zoom documentation in the handbook, you can do that. However, I think we (CT) decided our goal was to cut back on the instructional content on our end because this adds to the maintenance load for our documentation when Zoom makes changes. So I think the most sustainable choice is to link only to instructions that are hosted and maintained by Zoom.

instructors/instructor-notes.md Outdated Show resolved Hide resolved
ndporter and others added 2 commits February 28, 2024 15:49
Fix punctuation in tips

Co-authored-by: Karen Word <[email protected]>
@ndporter ndporter merged commit 38df3cd into main Feb 28, 2024
3 checks passed
@ndporter ndporter deleted the karenword-trainer-notes branch February 28, 2024 20:52
github-actions bot pushed a commit that referenced this pull request Feb 28, 2024
Auto-generated via {sandpaper}
Source  : 38df3cd
Branch  : main
Author  : Nathaniel D Porter <[email protected]>
Time    : 2024-02-28 20:52:21 +0000
Message : Merge pull request #1573 from carpentries/karenword-trainer-notes

Multiples updates to instructor-notes.md to reflect current practices and recommendations.
github-actions bot pushed a commit that referenced this pull request Feb 28, 2024
Auto-generated via {sandpaper}
Source  : 4c863c0
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2024-02-28 20:53:18 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : 38df3cd
Branch  : main
Author  : Nathaniel D Porter <[email protected]>
Time    : 2024-02-28 20:52:21 +0000
Message : Merge pull request #1573 from carpentries/karenword-trainer-notes

Multiples updates to instructor-notes.md to reflect current practices and recommendations.
github-actions bot pushed a commit that referenced this pull request Mar 5, 2024
Auto-generated via {sandpaper}
Source  : 4c863c0
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2024-02-28 20:53:18 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : 38df3cd
Branch  : main
Author  : Nathaniel D Porter <[email protected]>
Time    : 2024-02-28 20:52:21 +0000
Message : Merge pull request #1573 from carpentries/karenword-trainer-notes

Multiples updates to instructor-notes.md to reflect current practices and recommendations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants