-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
added employee badges to members and recognition page #6304
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Aviral Jain <[email protected]>
🚀 Preview for commit 84afdb1 at: https://67cee93babd2b40a44442371--layer5.netlify.app |
Signed-off-by: Aviral Jain <[email protected]>
🚀 Preview for commit 293d48c at: https://67cef310b68a7d009242342d--layer5.netlify.app |
@0x-74 please move from role to badges |
@vishalvivekm what do you mean? |
@0x-74 these changes, while have added the employee badge to handbook's recognition section, haven't made the badge available on /community/members page i.e. it can't be awarded to a particular profile like other badges, such as Meshery badge. Instead, it's in Roles, ( both Maintainer and Employee roles use the same 5 icon), but if you come to think of it, an individual can be both a maintainer and an employee e.g. Amit Amruitya, how should we account for this situation? Any thoughts? Another point to consider: should we add a flare to an employee's profile, similar to what we do for MeshMates? |
@0x-74 thoughts ? |
hey @vishalvivekm sorry for the late reply! i am actually able to add it as a badge so i dont quite understand that point |
@0x-74 adding it as an agenda item to the meeting minutes. |
🚀 Preview for commit c4ce41c at: https://67d7f8c5e50e2c7fbaad238f--layer5.netlify.app |
Hey @vishalvivekm i couldn't attend the meeting due to a meeting at my company, is it possible to connect elsewhere or can I get some task updates right here? |
Description
This PR fixes #6277
Notes for Reviewers
Signed commits