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

Revise roles page #271

Merged
merged 7 commits into from
Aug 22, 2023
Merged

Revise roles page #271

merged 7 commits into from
Aug 22, 2023

Conversation

vcarl
Copy link
Member

@vcarl vcarl commented Aug 21, 2023

No description provided.

@what-the-diff
Copy link

what-the-diff bot commented Aug 21, 2023

PR Summary

  • Title Change for key Contributors

    • not quite The title for MVPs is renamed to "Star Helpers" aligning better with the role played by these important contributors.
    • The description for "Star Helpers" is updated with additional information about quarterly reviews and recognition for their contributions.
  • Introduction of 'MVP Emeritus' Title

    • no The title "MVP" has been redefined to "MVP Emeritus".
    • The updated description for "MVP Emeritus" includes clarification on their early impact and lack of formalized process.
  • Clarifications on Moderators and Admins Roles

    • The titles for "Moderators" and "Admins" remain the same.
    • Both sections have received updates in their descriptions. The responsibilities of "Moderators" and "Admins" within the system are now better defined and clearer.

@netlify
Copy link

netlify bot commented Aug 21, 2023

Deploy Preview for reactiflux ready!

Name Link
🔨 Latest commit 5cbb51d
🔍 Latest deploy log https://app.netlify.com/sites/reactiflux/deploys/64e5288e58ea850007272987
😎 Deploy Preview https://deploy-preview-271--reactiflux.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@nickserv nickserv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy is good, but I think we should keep the links so non-members can contact us (for example if there's a question about promotion policies or an unban request from a friend of a banned member) and so members can reuse sessions and settings (including accessibility settings that the user may rely on) from the native client.

@vcarl
Copy link
Member Author

vcarl commented Aug 22, 2023

Non-members can use the other contact means, In real terms we've had 127 people, of our 220,000, ever join through here. The link works better for existing members and I'm not concerned about losing this as a source of entrants
image

@nickserv
Copy link
Member

nickserv commented Aug 22, 2023

Fair, but I'm still concerned about the latter issue:

members can reuse sessions and settings (including accessibility settings that the user may rely on) from the native client

In my experience most users are primarily using the native Discord client, and therefore direct links are more difficult to open for users that aren't logged in or don't have their preferred settings on web. This likely affects most users to an extent, as Discord doesn't attempt to sync these with the native client.

Alternatively we could also solve these issues using the discord protocol, which unlike invite links will directly open the native client, but not the web client.

@vcarl
Copy link
Member Author

vcarl commented Aug 22, 2023

Oh word, it's just a direct swap like discord://discord.com/channels/102860784329052160/106168778013822976? Yeah that's easy. I'll add this, I think with the web URLs linked as a backup

@vcarl vcarl merged commit 8859a9e into main Aug 22, 2023
4 of 6 checks passed
@vcarl vcarl deleted the vcarl-patch-1 branch August 22, 2023 21:41
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.

2 participants