Fix syscom roles #347

Merged
r389li merged 1 commits from raymo-fix-syscom-roles into main 2021-10-13 16:28:08 -04:00
Owner
No description provided.
r389li added the
Content
label 2021-10-13 15:55:08 -04:00
r389li added 1 commit 2021-10-13 15:55:10 -04:00
continuous-integration/drone/push Build is passing Details
25ac395f05
Fix syscom roles
a3thakra reviewed 2021-10-13 16:16:31 -04:00
@ -11,3 +11,3 @@
{
"name": "Raymond Li",
"role": "Member"
"role": ""
Owner

Any reason the "Member" role is not acceptable? I think the "" role looks a bit off UI wise, since everyone on other teams have a role.

I'm other to other words as well if you don't like "Member" :)

![](https:

Any reason the "Member" role is not acceptable? I think the "" role looks a bit off UI wise, since everyone on other teams have a role. I'm other to other words as well if you don't like "Member" :) ![](https:
Author
Owner

It sounds like a regular club member, and we don't really have roles on Syscom (but it's good to have the alumni labelled as such).

It sounds like a regular club member, and we don't really have roles on Syscom (but it's good to have the alumni labelled as such).
Author
Owner

Hmm or we could change the alumni's profile pics to "Alumni" and get rid of those roles, that way only Max has a role, would that look better?

Hmm or we could change the alumni's profile pics to "Alumni" and get rid of those roles, that way only Max has a role, would that look better?
Owner

on second thought, I'm fine with "", i think thats better considering there arent many roles in syscom.

on second thought, I'm fine with "", i think thats better considering there arent many roles in syscom.
Owner
No description provided.
a3thakra approved these changes 2021-10-13 16:22:26 -04:00
a3thakra left a comment
Owner

lgtm

lgtm
Owner

I'm good with what it looks like rn, I take my previous statement back haha 🙈

I'm good with what it looks like rn, I take my previous statement back haha 🙈
Author
Owner

Lol ok I'll merge it then.

Lol ok I'll merge it then.
r389li merged commit 1ce1d6a087 into main 2021-10-13 16:28:08 -04:00
r389li referenced this issue from a commit 2021-10-13 16:28:09 -04:00
a3thakra deleted branch raymo-fix-syscom-roles 2021-10-13 16:30:57 -04:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: www/www-new#347
No description provided.