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

Add Event: Dec 2, 2021 DEI Media Club #594

Merged
merged 6 commits into from
Nov 16, 2021
Merged

Conversation

mrmundt
Copy link
Contributor

@mrmundt mrmundt commented Nov 15, 2021

Description

Add event page for Dec 2, 2021 DEI Media Club.

Checklist:

  • I have posted the link for the PR in the usrse slack (#website) to ask for reviewers
  • I have previewed changes locally

cc @usrse-maintainers

Co-authored-by: Vanessasaurus <814322+vsoch@users.noreply.github.com>
@vsoch
Copy link
Member

vsoch commented Nov 15, 2021

One final comment for discussion!

  1. Do we want the zoom link open right there on the site (e.g., zoom bombing?)
  2. If so, should we make it a link proper?

image

@cosden
Copy link
Member

cosden commented Nov 16, 2021

One final comment for discussion!

  1. Do we want the zoom link open right there on the site (e.g., zoom bombing?)
  2. If so, should we make it a link proper?

image

  1. I think it's fine if the host is ok with it. It makes it more accessible, and until we have a problem, let's go for more public.
  2. Agreed, and just suggested it.

mrmundt and others added 2 commits November 16, 2021 07:47
Co-authored-by: Ian Cosden <cosden@users.noreply.github.com>
Co-authored-by: Ian Cosden <cosden@users.noreply.github.com>
@mrmundt
Copy link
Contributor Author

mrmundt commented Nov 16, 2021

All concerns addressed, I believe!

@vsoch
Copy link
Member

vsoch commented Nov 16, 2021

@mrmundt you were unlucky that a job entry has 404'd - what we usually do is ask the PR opener to remove the failing job, in this case it's Calico: https://www.calicolabs.com/careers?gh_jid=4962448002. If you don't have the bandwidth and want me to, I can also offer to do this and update the PR here.

@vsoch vsoch merged commit 5fa9f8c into USRSE:main Nov 16, 2021
@mrmundt mrmundt deleted the event/dei-media-club branch September 7, 2022 19:56
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.

None yet

4 participants