Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add MultiTenant #310

Merged
merged 6 commits into from
Mar 23, 2021
Merged

Add MultiTenant #310

merged 6 commits into from
Mar 23, 2021

Conversation

JieXiong9119
Copy link
Contributor

See proposal
Implementation in schema added

1. Changed `TotalExteriorRoofArea` to `TotalRoofArea`.
2. Created new simple type for `nonNegativeDecimal` and applied for the added element.
Copy link
Contributor

@macintoshpie macintoshpie left a comment

Choose a reason for hiding this comment

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

Can this be captured by an OccupancyClassification?

@corymosiman12
Copy link
Contributor

There are probably many different occupancy classifications that might be multi-tenant (office, mixed commercial, retail, etc.), but even within those it's not always guaranteed. This makes it clear.

@nllong nllong merged commit f9e48b4 into develop Mar 23, 2021
@nllong nllong deleted the proposals/multi-tenant branch March 23, 2021 20:13
@JieXiong9119 JieXiong9119 changed the title Proposals/multi tenant Add MultiTenant Sep 27, 2021
@JieXiong9119 JieXiong9119 added the Schema: General General update to BuildingSync label Sep 28, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature Adding new functionality to BuildingSync Non-breaking Change Schema: General General update to BuildingSync UDFs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants