Skip to content

Conversation

@emgrol
Copy link
Contributor

@emgrol emgrol commented Jan 22, 2020

No description provided.

Emily Olshefski added 2 commits January 21, 2020 13:49
- Formatting issues start at line 197
    - comments not showing, can't find error
- ? where I didn't know what to put for that section (feel free to edit/suggest correct content)
@emgrol emgrol added the WIP label Jan 22, 2020
@emgrol emgrol requested review from axelsrz and mmiele January 22, 2020 20:35
@emgrol emgrol self-assigned this Jan 22, 2020
@emgrol emgrol removed the WIP label Jan 23, 2020
@emgrol emgrol marked this pull request as ready for review January 23, 2020 19:10
Copy link
Contributor

@mmiele mmiele left a comment

Choose a reason for hiding this comment

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

It looks good.
When you say """ (not sure where to put this information)
Containers are typically leaf nodes on the stack but the dev is free to push other dialogs …
I would suggest to use this syntax:
.. note::
Containers are typically leaf nodes on the stack but the dev is free to push other dialogs …
See https://review.docs.microsoft.com/en-us/help/onboard/admin/reference/python/documenting-api?branch=master#note

@tracyboehrer tracyboehrer merged commit 2217314 into python-API-ref-docs Jan 27, 2020
@emgrol emgrol deleted the emolsh/api-ref-docs-componentdialog branch February 5, 2020 17:07
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.

4 participants