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

feat: import queries directly from source-iotsitewise #63

Merged
merged 1 commit into from
Feb 19, 2022

Conversation

boweihan
Copy link
Contributor

@boweihan boweihan commented Feb 18, 2022

Overview

Export and use query methods for sitewise directly from the source-iotsitewise module. Set up a build system for source-iotsitewise.

This is effectively a passthrough export at the moment because fully decoupling sitewise from the app kit core turned out to be a much larger task.

Tests

https://github.com/boweihan/iot-app-kit/actions/runs/1866382245

Legal

This project is available under the Apache 2.0 License.

"pack": "yarn pack"
},
"dependencies": {
"@iot-app-kit/core": "0.0.1",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These aren't used at the moment but are the core deps that will be needed once we move sitewise specific files in here.

@boweihan boweihan force-pushed the source-iotsitewise-alias branch 2 times, most recently from db49f6c to eaf9b2b Compare February 18, 2022 20:15
@@ -0,0 +1,3 @@
import { query, datamodule } from '@iot-app-kit/core';

export { query, datamodule };
Copy link
Contributor

Choose a reason for hiding this comment

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

could you move the query namespace file? seems like if something isn't moved this could go backwards.

@boweihan boweihan merged commit 9807c69 into awslabs:main Feb 19, 2022
TheEvilDev pushed a commit that referenced this pull request Nov 2, 2022
This was referenced Nov 2, 2022
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