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

fix(model): Add global_name to CurrentUser #2412

Merged

Conversation

Erk-
Copy link
Member

@Erk- Erk- commented Jan 28, 2025

Add global_name to CurrentUser

Closes: #2411

@github-actions github-actions bot added c-model Affects the model crate t-fix Fixes a bug in the library labels Jan 28, 2025
@@ -50,6 +50,9 @@ pub struct CurrentUser {
/// All flags on a user's account.
#[serde(skip_serializing_if = "Option::is_none")]
pub flags: Option<UserFlags>,
/// User's global display name, if set. For bots, this is the application name
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/// User's global display name, if set. For bots, this is the application name
/// User's global display name, if set. For bots, this is the application name.

@github-actions github-actions bot added c-cache Affects the cache crate c-standby Affects the standby crate labels Jan 28, 2025
@Erk- Erk- force-pushed the fix/model/add-missing-global-name-for-current-user branch from e1be18a to ccefdf7 Compare January 30, 2025 21:07
@HTGAzureX1212 HTGAzureX1212 merged commit 4507b77 into main Feb 1, 2025
9 checks passed
@HTGAzureX1212 HTGAzureX1212 deleted the fix/model/add-missing-global-name-for-current-user branch February 1, 2025 11:22
viztea pushed a commit to eara-bot/twilight that referenced this pull request Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-cache Affects the cache crate c-model Affects the model crate c-standby Affects the standby crate t-fix Fixes a bug in the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CurrentUser is missing global_name
4 participants