Missing supported data feed sources on DataFeedSource docs

[Gyeonghun-Park]

There should be AzureApplicationInsightsDataFeedSource, AzureBlobDataFeedSource, AzureCosmosDbDataFeedSource, AzureDataExplorerDataFeedSource, AzureDataLakeStorageDataFeedSource, AzureEventHubsDataFeedSource, AzureTableDataFeedSource, InfluxDbDataFeedSource, LogAnalyticsDataFeedSource, MongoDbDataFeedSource, MySqlDataFeedSource, PostgreSqlDataFeedSource and SqlServerDataFeedSource, not empty bulleted list.

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

• ID: 48efe551-7921-222e-d0bd-25dbdda967b1
• Version Independent ID: 9efc5e45-b035-7e7c-a0ce-399159b8b021
• Content: DataFeedSource Class (Azure.AI.MetricsAdvisor.Administration) - Azure for .NET Developers
• Content Source: xml/Azure.AI.MetricsAdvisor.Administration/DataFeedSource.xml
• Service: azure
• GitHub Login: @rloutlaw
• Microsoft Alias: routlaw

[jsquire]

Thank you for your feedback. Tagging and routing to the team member best able to assist.

[jsquire]

@kinelski: The root cause for this is that the <list> tags are malformed in the class summary area, here. The <list> must have a child element of <description> and, optionally, may have a child <term> as well. (ref)

We'll need to wrap the <see> elements for them to render correctly. (example)

[Gyeonghun-Park]

Hello @jsquire,
Can I add <description> tag to DataFeedSource.cs file and make a PR for this issue?

azure-sdk-for-net/sdk/metricsadvisor/Azure.AI.MetricsAdvisor/src/Models/DataFeed/DataFeedSource.cs

Lines 13 to 27 in 4efd92b

	/// <list type="bullet">
	/// <item><see cref="AzureApplicationInsightsDataFeedSource"/></item>
	/// <item><see cref="AzureBlobDataFeedSource"/></item>
	/// <item><see cref="AzureCosmosDbDataFeedSource"/></item>
	/// <item><see cref="AzureDataExplorerDataFeedSource"/></item>
	/// <item><see cref="AzureDataLakeStorageDataFeedSource"/></item>
	/// <item><see cref="AzureEventHubsDataFeedSource"/></item>
	/// <item><see cref="AzureTableDataFeedSource"/></item>
	/// <item><see cref="InfluxDbDataFeedSource"/></item>
	/// <item><see cref="LogAnalyticsDataFeedSource"/></item>
	/// <item><see cref="MongoDbDataFeedSource"/></item>
	/// <item><see cref="MySqlDataFeedSource"/></item>
	/// <item><see cref="PostgreSqlDataFeedSource"/></item>
	/// <item><see cref="SqlServerDataFeedSource"/></item>
	/// </list>

[jsquire]

Hi @Gyeonghun-Park. Thank you for offering to help improve the Azure SDK experience! Generally, we love contributions and this doesn't appear to be generated code - but I'd like to ask @kinelski for his thoughts. I don't want to see you spend time if this is something that needs an upstream change in the swagger.

[kinelski]

Hello @Gyeonghun-Park. Feel free to create a PR with the fix! I'm assigning this issue to you. Let me know if you have any questions.

[kinelski]

Fixed by @Gyeonghun-Park in #24764.