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

Many Summaries added. #400

Merged
merged 3 commits into from
Jun 23, 2024
Merged

Many Summaries added. #400

merged 3 commits into from
Jun 23, 2024

Conversation

Mahsaap
Copy link
Member

@Mahsaap Mahsaap commented Jun 23, 2024

Summaries added.
Removed some unused usings.
Changed many namespaces to ; instead of {}

Note: With some other changes I have in my WIP, I was missing 2 files to allow these to succeed in #399 .

Comment on lines 352 to 354
/// <summary>
///
/// </summary>
Copy link

Choose a reason for hiding this comment

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

empty summary


/// <summary>
/// The amount of time, in seconds, that users must wait between sending messages.
///
Copy link

Choose a reason for hiding this comment

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

extra new line

public Dictionary<string, string> LocalizationDescriptions { get; protected set; }
}
/// <summary>
/// An ID that identifies this tag.
Copy link

Choose a reason for hiding this comment

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

extra tab

Comment on lines 7 to 11
/// <summary>
/// /// <summary>
/// An organization’s entitlement that has been granted to a game, a user, or both.
/// </summary>
/// </summary>
Copy link

Choose a reason for hiding this comment

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

double summary

public string Id { get; protected set; }

/// <summary>
/// The UTC date and time (in RFC3339 format) of the transaction.
Copy link

Choose a reason for hiding this comment

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

extra tab

@@ -13,15 +13,13 @@ namespace TwitchLib.Api.Helix
public class ContentClassificationLabels : ApiBase
{
/// <summary>
/// <para></para>
///
Copy link

Choose a reason for hiding this comment

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

empty summary

public int RetryAfter { get; protected set; }
}
/// <summary>
/// The number of seconds you must wait before running another commercial.
Copy link

Choose a reason for hiding this comment

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

extra space

/// <summary>
/// A Boolean value that determines whether chat messages must contain only emotes.
/// </summary>
///
Copy link

Choose a reason for hiding this comment

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

extra new line

@@ -2,17 +2,27 @@

namespace TwitchLib.Api.Helix.Models.EventSub.Conduits.Shards;

/// <summary>
///
Copy link

Choose a reason for hiding this comment

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

empty summary

public string ModeratorId { get; protected set; }

/// <summary>
/// Login of the moderator who initiated the ban.
Copy link

Choose a reason for hiding this comment

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

extra tab

@Mahsaap Mahsaap merged commit 978edaf into TwitchLib:dev Jun 23, 2024
1 check passed
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