-
Notifications
You must be signed in to change notification settings - Fork 151
Building Onboarding PR #1560
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
Building Onboarding PR #1560
Conversation
Merging master into current branch to sync updates
@shambergoldstein Few comments back for you, but should be ready for review! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
generally, we want to model the devices with types that represent how they physically work in the real world regardless of what points are bacnet available. anything not available we should just mark missing
@ccquigley Ready for your review! Just wrapped up the other comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please address validator errors
@hhunterr71 the validation is still failing, please remove the duplicate fields on ECOND |
@shambergoldstein Fixed |
PR affects the following types:
ABSTRACT, AHU, CDWS, CH, CHWS, CT, DHWT, FAN, FCU, HWS, MAU, SENSOR,UH.