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

Add examples in Wiki Commands + minor table format edit #591

Merged
merged 3 commits into from
May 10, 2019

Conversation

atbagga
Copy link
Collaborator

@atbagga atbagga commented May 10, 2019

Please make sure the code is following contribution guidelines in CONTRIBUTING.md

@atbagga atbagga self-assigned this May 10, 2019
@@ -215,7 +215,7 @@ def transform_wiki_page_table_output(result):
def _transform_wiki_page_row(row):
table_row = OrderedDict()
table_row['ETag'] = row['eTag']
table_row['Git Path'] = row['page']['gitItemPath']
table_row['Page Path'] = '\'{}\''.format(row['page']['path'])
Copy link
Contributor

Choose a reason for hiding this comment

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

just curious, why are we doing this in table output specifically?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

image

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

User needs to use the page path in the command not git item path. So I should have always put this in the table output in the first place. See sample above.

@atbagga atbagga closed this May 10, 2019
@atbagga atbagga reopened this May 10, 2019
@atbagga atbagga merged commit 3a01c5b into master May 10, 2019
@atbagga atbagga deleted the users/atbagga/wikipagepath branch May 23, 2019 08:19
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.

3 participants