Skip to content

In owner invite messages, use crates.io username - #14581

Merged
Turbo87 merged 1 commit into
rust-lang:mainfrom
carols10cents:username-owner-messages
Sep 4, 2026
Merged

In owner invite messages, use crates.io username#14581
Turbo87 merged 1 commit into
rust-lang:mainfrom
carols10cents:username-owner-messages

Conversation

@carols10cents

Copy link
Copy Markdown
Member

These are messages that a user sees when they've invited other users to own a crate. Right now, there shouldn't be any confusion because users.username should always equal users.gh_login, but I've added "Crates.io user" to the message to hopefully clarify for the future when we have crates.io usernames independent of GitHub usernames.

Extracted from #14575; will probably conflict with #14213 a little bit.

These are messages that a user sees when they've invited other users to
own a crate. Right now, there shouldn't be any confusion because
`users.username` should always equal `users.gh_login`, but I've added
"Crates.io user" to the message to hopefully clarify for the future when
we have crates.io usernames independent of GitHub usernames.

@moskirathe moskirathe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i'll update #14213 to resolve the minor conflict this introduces. Should be easy enough to propagate the username type (crates.io or github currently) so that its included in the invite messages.

View changes since this review

@Turbo87
Turbo87 merged commit 5477be9 into rust-lang:main Sep 4, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants