Skip to content

Adds 'pa app get', solving #2110#2172

Closed
appieschot wants to merge 1 commit intopnp:mainfrom
appieschot:feature/cmd-2110
Closed

Adds 'pa app get', solving #2110#2172
appieschot wants to merge 1 commit intopnp:mainfrom
appieschot:feature/cmd-2110

Conversation

@appieschot
Copy link
Copy Markdown
Member

Adds 'pa app get', solving #2110

@waldekmastykarz
Copy link
Copy Markdown
Member

Awesome! Let's get it reviewed asap 👏

Copy link
Copy Markdown
Member

@waldekmastykarz waldekmastykarz left a comment

Choose a reason for hiding this comment

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

Almost perfect 😊 Awesome work! 👏

Get-FlowOwnerRole|Microsoft.PowerApps.PowerShell|
Get-FlowRun|Microsoft.PowerApps.PowerShell|[flow run list](../cmd/flow/run/run-list.md), [flow run get](../cmd/flow/run/run-get.md)
Get-PowerApp|Microsoft.PowerApps.PowerShell|
Get-PowerApp|Microsoft.PowerApps.PowerShell|[pa app list](../cmd/app/app/app-list.md), [pa app get](../cmd/app/app/app-get.md)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Paths to MD files are incorrect, should've been ../cmd/pa/app/app-list.md and ../cmd/pa/app/app-get.md

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sorry for that! Guess I had thick fingers ;-)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Very minor thing, don't worry about it. Just before I ran docs I was convinced it was a 💯 commit 💪

@waldekmastykarz
Copy link
Copy Markdown
Member

Merged manually. Thank you! 👏

@appieschot appieschot deleted the feature/cmd-2110 branch February 26, 2021 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants