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

{powerbi} HTML tags are rendering as empty parameter values in reference #29619

Closed
dbradish-microsoft opened this issue Jul 31, 2024 · 4 comments · Fixed by #29898
Closed

{powerbi} HTML tags are rendering as empty parameter values in reference #29619

dbradish-microsoft opened this issue Jul 31, 2024 · 4 comments · Fixed by #29898
Assignees
Labels
Azure CLI Team The command of the issue is owned by Azure CLI team feature-request PowerBI

Comments

@dbradish-microsoft
Copy link
Contributor

Describe the bug

Problem to solve:

HTML tags in source code are causing build warnings in MicrosoftDocs/azure-docs-cli. More importantly, customer-facing reference docs are not instantiating properly -- parameter values formatted with `<>` are missing. Here's an example:
image

Requested solution:

These values should be escaped with a backtick. Here's an example from az webpubsub.
image

Published reference guidelines

Guidelines for Parameters used in examples

Related GitHub issues:

This is the same issue as these GitHub issues.

List of HTML tag instances that need update:

File name Build error
latest/docs-ref-autogen/powerbi/embedded-capacity.yml HTML tag 'indexToRemove' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
latest/docs-ref-autogen/powerbi/embedded-capacity.yml HTML tag 'value' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

Related command

az powerbi

Errors

n/a

Issue script & Debug output

n/a

Expected behavior

n/a

Environment Summary

Azure CLI 2.62.0

Additional context

No response

@dbradish-microsoft dbradish-microsoft added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Jul 31, 2024
@yonzhan
Copy link
Collaborator

yonzhan commented Jul 31, 2024

Thank you for opening this issue, we will look into it.

@yonzhan yonzhan added PowerBI Service Attention This issue is responsible by Azure service team. labels Aug 1, 2024
Copy link
Contributor

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @laurent-mic, @ayeshurun.

@ayeshurun
Copy link

@yonzhan @dbradish-microsoft the remove property is defined by the core azure cli package - https://github.com/search?q=repo%3AAzure%2Fazure-cli+%22Remove+a+property+or+an+element+from+a+list.%22&type=code

@AllyW AllyW removed the Service Attention This issue is responsible by Azure service team. label Sep 12, 2024
@AllyW
Copy link
Member

AllyW commented Sep 12, 2024

Thanks @ayeshurun @dbradish-microsoft for reporting and debugging, these disallowed indexToRemove and value tags are defined in cli.core and would be fixed this sprint.

@AllyW AllyW self-assigned this Sep 12, 2024
@AllyW AllyW added this to the September 2024 (2024-10-08) milestone Sep 12, 2024
@yonzhan yonzhan added Azure CLI Team The command of the issue is owned by Azure CLI team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that feature-request and removed bug This issue requires a change to an existing behavior in the product in order to be resolved. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure CLI Team The command of the issue is owned by Azure CLI team feature-request PowerBI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants