Username Docs Updates #6218
Merged
Username Docs Updates #6218
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: Florian Spieß <business@minn.dev>
Co-authored-by: Néstor Pérez <25409753+prryplatypus@users.noreply.github.com>
docs/Change_Log.md
Outdated
|
|
||
| ### Default avatars | ||
|
|
||
| For users with migrated accounts, default avatar URLs will be based on the user ID instead of the discriminator. The URL can now be calculated using `(user_id >> 22) % 6`. |
There was a problem hiding this comment.
Is % 6 is a typo or was this just changed randomly?
There was a problem hiding this comment.
this was changed. pink avatars now wowwwww yay fun 🌸
There was a problem hiding this comment.
Do users with discriminators still use discriminator % 5?
There was a problem hiding this comment.
yeah, for non-migrated users it's still % 5. i can add a note about that until all users are migrated.
shaydewael
force-pushed
the
username-updates
branch
from
b7a4d80
to
8c74893
Compare
Co-authored-by: Florian Spieß <business@minn.dev>
Co-authored-by: Florian Spieß <business@minn.dev>
shaydewael
force-pushed
the
username-updates
branch
from
722485f
to
7379fc0
Compare
TTtie
added a commit
to projectdysnomia/dysnomia
that referenced
this pull request
Jun 7, 2023
|
https://github.com/discord/discord-api-docs/blob/main/docs/resources/User.md Line 30: This is outdated now, right? the username with its new rules should be unique now. |
The username won't be unique until all users are migrated. Since bots aren't even getting migrated yet, it's definitely still accurate. |
2 tasks
13 tasks
ooliver1
reviewed
Jun 8, 2023
| @@ -28,7 +28,8 @@ There are other rules and restrictions not shared here for the sake of spam and | |||
| | ------------- | --------- | ---------------------------------------------------------------------------------------------------- | --------------------- | | |||
| | id | snowflake | the user's id | identify | | |||
| | username | string | the user's username, not unique across the platform | identify | | |||
| | discriminator | string | the user's 4-digit discord-tag | identify | | |||
7 tasks
lukellmann
added a commit
to kordlib/kord
that referenced
this pull request
Jun 10, 2023
mccoderpy
added a commit
to mccoderpy/discord.py-message-components
that referenced
this pull request
Jun 11, 2023
discord/discord-api-docs#6218 Signed-off-by: Mathieu Corsham <McCuber04@outlook.de>
4 tasks
9 tasks
Nihlus
added a commit
to Remora/Remora.Discord
that referenced
this pull request
Jun 17, 2023
VelvetToroyashi
pushed a commit
to VelvetToroyashi/Remora.Discord
that referenced
this pull request
Jun 17, 2023
TTtie
added a commit
to projectdysnomia/dysnomia
that referenced
this pull request
Jun 18, 2023
* initial unique username system support * fix up wrong ordering * add a newline between new member getter * migrated users have 6 default avatars now ref: discord/discord-api-docs#6218 (comment) * emit old globalName in userUpdate * whoops: global_name -> globalName * copy note about single "0" discrim to Member.js
TTtie
added a commit
to projectdysnomia/dysnomia
that referenced
this pull request
Jul 7, 2023
* initial unique username system support * fix up wrong ordering * add a newline between new member getter * migrated users have 6 default avatars now ref: discord/discord-api-docs#6218 (comment) * emit old globalName in userUpdate * whoops: global_name -> globalName * copy note about single "0" discrim to Member.js
TTtie
added a commit
to projectdysnomia/dysnomia
that referenced
this pull request
Aug 10, 2023
* initial unique username system support * fix up wrong ordering * add a newline between new member getter * migrated users have 6 default avatars now ref: discord/discord-api-docs#6218 (comment) * emit old globalName in userUpdate * whoops: global_name -> globalName * copy note about single "0" discrim to Member.js
shaydewael
added a commit
to Jupith/discord-api-docs
that referenced
this pull request
May 14, 2024
* start username docs * default avatar * Update docs/Reference.md Co-authored-by: Florian Spieß <business@minn.dev> * Update docs/Change_Log.md Co-authored-by: Néstor Pérez <25409753+prryplatypus@users.noreply.github.com> * fix docs * reorder * clarify migrated/legacy * small changelog clarification * Update docs/Change_Log.md Co-authored-by: Florian Spieß <business@minn.dev> * Update docs/Reference.md Co-authored-by: Florian Spieß <business@minn.dev> * oops * great --------- Co-authored-by: Florian Spieß <business@minn.dev> Co-authored-by: Néstor Pérez <25409753+prryplatypus@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ok this is just updated #6130 but the branch wasn't syncing correctly so I'm just creating new PR oop.
this no longer includes info specific to bots
edit: ok the PR thing was just related to a Github issue i need to calm down