Add an email to SR2025 secondary contacts #193
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
So that they know what's going on. Also means we'll find out sooner than later if their email address doesn't work.
So that the secondary contact has a bit more info about what the primary contact is expected to do.
Mostly so that the secondary contact is also aware of the kit loan.
samjmartin04
previously approved these changes
Sep 15, 2024
Co-authored-by: Sam Martin <[email protected]>
PeterJCLaw
commented
Sep 15, 2024
Comment on lines
16
to
18
| As [team supervisor][team-supervisor], $TODO_PRIMARY_CONTACT_GIVEN_NAME is | ||
| responsible for your competitors as well as the [kit][kit] which we loan to each | ||
| team. Our communication with your team will also all be sent to them. |
There was a problem hiding this comment.
There's two things I'm trying to convey here:
- awareness of the kit loan
- awareness that if the primary contact disappears that we'll chase the secondary for the kit and for looking after the competitors
I almost want to make it clear that the secondary is thus also responsible for the kit, but I'm not sure if there's a way to do that without it getting really complicated.
sedders123
previously approved these changes
Sep 18, 2024
PeterJCLaw
commented
Sep 18, 2024
PeterJCLaw
force-pushed
the
sr2025-secondary-contacts
branch
from
fa8f4bd to
6b1d83d
Compare
GMail's mail merge doesn't support CC (either fixed or templated), hence needing to do this ourselves.
PeterJCLaw
force-pushed
the
sr2025-secondary-contacts
branch
from
6b1d83d to
32651ed
Compare
2 tasks
|
Going to merge since this was sent. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
So that they know what's going on. Also means we'll find out sooner than later if their email address doesn't work.
This also includes links to our docs around the kit loan and the team supervisor responsibilities.
Would be great to merge srobo/docs#614 before we send this, but it's not a blocker.
Fixes srobo/tasks#1467
Checklist
toandsubject)