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

[feature + chore] adding DescribeAssetModel integration #80

Merged
merged 2 commits into from
Feb 3, 2021

Conversation

bymattoa
Copy link
Contributor

@bymattoa bymattoa commented Feb 3, 2021

  • [feature] Added handler for Sitewise::DescribeAssetModel API. This is being added primarily to support efficient template variables: Variables: add variable support #77
  • [chore] Refactored api handler functions into a separate submodule of sitewise. Primarily for better code organization and to reduce the odds of circular dependencies going forward

Screen Shot 2021-02-03 at 12 02 49 PM

@bymattoa
Copy link
Contributor Author

bymattoa commented Feb 3, 2021

@ryantxu - should be able to use this to wrap up #77

@ryantxu ryantxu self-requested a review February 3, 2021 20:40
@ryantxu ryantxu merged commit 338b4ea into main Feb 3, 2021
@ryantxu ryantxu deleted the describe-asset-model branch February 3, 2021 20:40
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.

None yet

2 participants