Skip to content

Show tags in list commands #57

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

Merged
merged 7 commits into from
Nov 17, 2021
Merged

Show tags in list commands #57

merged 7 commits into from
Nov 17, 2021

Conversation

polldo
Copy link
Contributor

@polldo polldo commented Nov 9, 2021

Motivation

We want to give users a way to see the tags on things/devices. Adding tags information in the list command should be good enough.

Change description

Add tags information in thing list and device list commands

Additional Notes

The tags Info function returns an error if a tag value is not a string. Alternatively the invalid tags can be simply skipped (at the moment the backend doesn't allow values different from string)

Reviewer checklist

  • PR address a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.
  • History is clean, commit messages are meaningful (see CONTRIBUTING.md) and are well formatted.

@polldo polldo requested review from eclipse1985 and glumia November 9, 2021 17:32
@polldo polldo merged commit 16536b0 into main Nov 17, 2021
@polldo polldo deleted the polldo/show-tags branch November 17, 2021 17:50
polldo pushed a commit that referenced this pull request Sep 2, 2022
Add tags information in 'thing list' and 'device list' commands
polldo pushed a commit that referenced this pull request Sep 2, 2022
Add tags information in 'thing list' and 'device list' commands
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