Skip to content

Use the Vanity Discord Invite#3077

Closed
Ombucha wants to merge 1 commit into
psf:mainfrom
Ombucha:patch-1
Closed

Use the Vanity Discord Invite#3077
Ombucha wants to merge 1 commit into
psf:mainfrom
Ombucha:patch-1

Conversation

@Ombucha
Copy link
Copy Markdown

@Ombucha Ombucha commented May 19, 2022

Description

This PR replaces the normal Discord invite, with the vanity one.

Checklist - did you ...

  • Add a CHANGELOG entry if necessary?
  • Add / update tests if necessary?
  • Add new / update outdated documentation?

@ichard26
Copy link
Copy Markdown
Collaborator

Hey thanks for opening a pull request! but ... is this necessary as the current invite is attached to the #black-formatter text channel specifically as far as I know, so in a way using the vanity link would be a downgrade. Are there any other benefits with the vanity URL that I am not aware of?

@ichard26 ichard26 added T: documentation Improvements to the docs (e.g. new topic, correction, etc) S: awaiting response Waiting for futher information from OP labels May 20, 2022
@Ombucha
Copy link
Copy Markdown
Author

Ombucha commented May 20, 2022

I don't think the vanity is attached to any specific channel, but it is easier to remember for users.

@ichard26 ichard26 added the ci: skip news Pull requests that don't need a changelog entry. label May 20, 2022
@ichard26
Copy link
Copy Markdown
Collaborator

Do people copy or look at the link directly often? It's embedded in a Markdown link so I doubt most users would actually care if it's easier to remember.

@Ombucha
Copy link
Copy Markdown
Author

Ombucha commented May 20, 2022

Yeah, I agree.

@Ombucha Ombucha closed this May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci: skip news Pull requests that don't need a changelog entry. S: awaiting response Waiting for futher information from OP T: documentation Improvements to the docs (e.g. new topic, correction, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants