-
Notifications
You must be signed in to change notification settings - Fork 61
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
[FEATURE] TikTok Shop API Upgrade to v202309 #31
Conversation
ThrowOnNotFoundRequestModelSerializationSampleFiles = true, | ||
ThrowOnNotFoundResponseModelSerializationSampleFiles = true | ||
}; | ||
new SourceFileAnalyzer(options).AssertNoIssues(); |
Check warning
Code scanning / CodeQL
Missing Dispose call on local IDisposable Warning test
ThrowOnNotFoundResponseModelTypes = true, | ||
ThrowOnNotFoundExecutingExtensionTypes = true | ||
}; | ||
new TypeDeclarationAnalyzer(options).AssertNoIssues(); |
Check warning
Code scanning / CodeQL
Missing Dispose call on local IDisposable Warning test
{ | ||
public static class Types | ||
{ | ||
public class Category : ProductGetCategoriesResponse.Types.Data.Types.Category |
Check notice
Code scanning / CodeQL
Class has same name as super class Note
{ | ||
public static class Types | ||
{ | ||
public class Attribute : ProductGetCategoryAttributesResponse.Types.Data.Types.Attribute |
Check notice
Code scanning / CodeQL
Class has same name as super class Note
...nce.TikTokGlobalShop/Models/Product/Global/Category/ProductGetGlobalCategoryRulesResponse.cs
Fixed
Show fixed
Hide fixed
public string[] OptionalRegionCodeList { get; set; } = default!; | ||
} | ||
|
||
public class SizeChart : ProductGetCategoryRulesResponse.Types.Data.Types.SizeChart |
Check notice
Code scanning / CodeQL
Class has same name as super class Note
{ | ||
public static class Types | ||
{ | ||
public class Category : ProductRecommendCategoriesResponse.Types.Data.Types.Category |
Check notice
Code scanning / CodeQL
Class has same name as super class Note
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
No description provided.