Add news for winter 2022 election results (Closes #380) #383

Merged
b72zhou merged 3 commits from w22-election-news into main 1 year ago
b72zhou commented 1 year ago
Collaborator

If Head of Discord, Heads of Design, Head of Reps are to be included, just notify me ~ 😃

If Head of Discord, Heads of Design, Head of Reps are to be included, just notify me ~ 😃
b72zhou added 1 commit 1 year ago
4ecbaa6709 add news for winter 2022 election results
b72zhou requested review from n3parikh 1 year ago
b72zhou requested review from a258wang 1 year ago
b72zhou requested review from j285he 1 year ago
b72zhou changed title from add news for winter 2022 election results to Add news for winter 2022 election results (Closes #380) 1 year ago
a258wang reviewed 1 year ago
- Treasurer: Eden Chan (e223chan)
- Sysadmin: Raymond Li (r389li)
<!-- -->
Owner

Is there a reason why we have this empty comment here at the end?

Is there a reason why we have this empty comment here at the end?
Poster
Collaborator

Oops this is just because I followed the format from Fall 2021. There was one empty comment at the end last time. Should I delete it?

Oops this is just because I followed the format from Fall 2021. There was one empty comment at the end last time. Should I delete it?
Owner

Ahh okay I see, it looks like it's just a visual dividing line for us when we're looking at the Markdown (it doesn't change how the news shows up on the website). Since all the past election announcements seem to have it, we can keep it - it's really not a big deal either way. 🙂

Ahh okay I see, it looks like it's just a visual dividing line for us when we're looking at the Markdown (it doesn't change how the news shows up on the website). Since all the past election announcements seem to have it, we can keep it - it's really not a big deal either way. 🙂
Collaborator

This might be an artifact of the XML to MD conversion process. It should be safe to delete.

This might be an artifact of the XML to MD conversion process. It should be safe to delete.
Owner

@b72zhou As per Neil's comment in #380, let's add the Head of Discord, Heads of Design, and Head of Reps as well:

  • Head of Discord: Andrew Wang (a268wang)
  • Heads of Design: Vivian Guo (v6guo) and Jenny Zhang (j2447zha)
  • Head of Reps: Amy Luo (a27luo)

You can follow the announcement format from Fall 2021 (but let's include the userids).

Please also wait until January 15 (EST) or later before merging this PR, so the news for Bootcamp mentor signups is not hidden.

@b72zhou As per Neil's comment in #380, let's add the Head of Discord, Heads of Design, and Head of Reps as well: - Head of Discord: Andrew Wang (a268wang) - Heads of Design: Vivian Guo (v6guo) and Jenny Zhang (j2447zha) - Head of Reps: Amy Luo (a27luo) You can follow the announcement format from Fall 2021 (but let's include the userids). Please also wait until January 15 (EST) or later before merging this PR, so the news for Bootcamp mentor signups is not hidden.
b72zhou added 1 commit 1 year ago
8b059ec81b add Heads of Discord/Design/Reps
a258wang approved these changes 1 year ago
a258wang left a comment
Owner

Looks good to me!

Looks good to me!
j285he approved these changes 1 year ago
n3parikh requested changes 1 year ago
Additionally, the following postions were appointed:
- Head of Discord: Andrew Wang (a268wang)
Owner

a268wang is the syscom member, not head of discord. I will find the correct user ID and get back to you.

a268wang is the syscom member, not head of discord. I will find the correct user ID and get back to you.
Owner

a284wang

a284wang
Owner

@b72zhou Would you be able to correct Andrew's username and then merge this branch? It might also be a good idea to write his name as "Andy Wang" to match the Meet the Team page. Thank you 🙂

@b72zhou Would you be able to correct Andrew's username and then merge this branch? It might also be a good idea to write his name as "Andy Wang" to match the Meet the Team page. Thank you 🙂
Poster
Collaborator

@n3parikh I'm not allowed to merge until you approve the change. Could you plz approve it?

@n3parikh I'm not allowed to merge until you approve the change. Could you plz approve it?
b72zhou added 1 commit 1 year ago
4f019554fb Edit Head of Discord
n3parikh approved these changes 1 year ago
b72zhou merged commit 78c751fce0 into main 1 year ago
b72zhou deleted branch w22-election-news 1 year ago

Reviewers

a258wang approved these changes 1 year ago
j285he approved these changes 1 year ago
n3parikh approved these changes 1 year ago
continuous-integration/drone/push Build is passing
The pull request has been merged as 78c751fce0.
Sign in to join this conversation.
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: www/www-new#383
Loading…
There is no content yet.