-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Admin commands documentation update for whiteboard to get migrated and originally created fluid boards #12685
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
Admin commands documentation update for whiteboard to get migrated and originally created fluid boards #12685
Conversation
…d originally created fluid whiteboards.
Learn Build status updates of commit d9f0278:
|
Admin commands documentation update to get migrated and originally created fluid whiteboards. |
Learn Build status updates of commit 7094535: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
@microsoft-github-policy-service agree company="Microsoft" |
Learn Build status updates of commit 7094535: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
@swagh-microsoft @SerdarSoysal We need to try to figure out why at least one of these articles doesn't show CommonParameters. My guess is that the Markdown doesn't precisely follow the schema, but I don't know yet. |
Learn Build status updates of commit 1620586: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Ah, I did not remember that the absence of CommonParameters from Learn articles was a long-standing and, so far, unsolved problem. |
Learn Build status updates of commit 8241c1c: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
### -Settings | ||
|
||
The object to use as Whiteboard Settings. Should be retrieved via Get-WhiteboardSettings | ||
|
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.
@swagh-microsoft @SerdarSoysal -Settings
must have a YAML code block to appear in the Learn article. At the moment, -Settings is absent from the preview, as seen by opening the View link for the article.
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.
After you've added an appropriate code block, please remove Back to submitter and reapply Sign off.
@swagh-microsoft @SerdarSoysal After you've added the code block, please remove Back to submitter and reapply Sign off. |
Learn Build status updates of commit f6fb4a3: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
@garycentric I am not able to update label. Updated PR with yaml settings. |
Learn Build status updates of commit 0435941: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
The list of objects is rendered as a single paragraph with word wrap when it is, by default, body text.
Learn Build status updates of commit f7c09f1: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
The code block was not rendered as such. I'm trying a table as a replacement.
Learn Build status updates of commit 7cdf228: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
I see that the page about Get-WhiteboardSettings has such a code block, so this ought to work (and not need to be rendered as a table).
Learn Build status updates of commit d67f245: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit fc97a3e: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit 99c0b8e: ❌ Validation status: errorsPlease follow instructions here which may help to resolve issue.
whiteboard/whiteboard-ps/whiteboard/Set-WhiteboardSettings.md
whiteboard/whiteboard-ps/whiteboard/index.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. For any questions, please:
|
This list of objects/properties is rendered as a single line with word wrap. Since this seems to be output from Get-WhiteboardSettings, and the page about Get-WhiteboardSettings contains this list (under "Examples")., I'm removing it here and adding a link to that article for "Get-WhiteboardSettings" in the description of "-Settings", which states, "Should be retrieved via Get-WhiteboardSettings."
Learn Build status updates of commit ee7b66e: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
@swagh-microsoft @SerdarSoysal Here's how that list of objects/properties looks in the page. I've removed it, because I didn't discover a way for it to be rendered well, and I added a link to Get-WhiteboardSettings, which seems to be the way that list is displayed. PubOps cannot approve content by add Sign off, but I'll merge this since you were addressing a problem with presentation. |
Learn Build status updates of commit 9d7ff33: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PubOps PR review, 2025-04-21, 11:25 AM