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.

Sign up for GitHub

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

Jump to bottom

[WIP] Add elements model #4689

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from
Open

[WIP] Add elements model #4689

wants to merge 11 commits into from

Conversation

devinbrady
Copy link

Any background context you want to provide?

Add the "elements" data model for components of buildings.

What's this PR do?

Add an model for elements, as well as API endpoints and frontend changes to display elements within each property.

How should this be manually tested?

What are the relevant tickets?

Screenshots (if appropriate)

@devinbrady devinbrady added DO NOT MERGE Feature Add this label to new features. This will be reflected in the change log when generated. ESTCP labels Jun 11, 2024
@axelstudios
- Added element tests
28e4d6d 
- Added element extra_data validator to prevent nested structures
- Added the ability to include enums in Swagger path arguments
- Fixed deprecated usages of assertDictContainsSubset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
DO NOT MERGE ESTCP Feature Add this label to new features. This will be reflected in the change log when generated.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@devinbrady @axelstudios