-
Notifications
You must be signed in to change notification settings - Fork 54
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
Add building groups functionality #4899
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Added inventory groups * fixed formatting * Add in AH * Groups Details page * Lint * rp edits * migration * migration order precommit * lint * fix tests --------- Co-authored-by: Hannah Eslinger <heslinge@nrel.gov> Co-authored-by: Ross Perry <ross.perry@rosss-mbp-2.lan> Co-authored-by: Ross Perry <ross.perry@Rosss-MBP-2.home>
* Added inventory groups * fixed formatting * Add in AH * Groups Details page * Lint * rp edits * fix migrations * add group buttons to inv detail * groups to look like labels/dcs * least common ancestor * add all alis to group modal * move lowest common to org view * development * migration * member list default * optimize lowest_common * functional pre same node constraint * validate inventorygroup for name and org * fe error handling on group create * mapping constraint, inventory and group must share ali * precommit * overhaul of the serializer and member list * add and remove button bugs * precommit * merge conflicts * merge conflicts * Fix `hiredis` not found error (#4821) Fix hiredis not found error * navigation bugs * navigation bugs * troubleshooting * id bug * migration order precommit * lint * fix tests * integrate with develop fix frontend tests lint * fix tests, simplify migration, formatting precommit * lint * dry mapping presave * group meters page cleanup cleanup cleanup broken ali typo meters page functional * fix test --------- Co-authored-by: Elizabeth <ebeers86@gmail.com> Co-authored-by: Hannah Eslinger <heslinge@nrel.gov> Co-authored-by: Ross Perry <ross.perry@rosss-mbp-2.lan> Co-authored-by: Ross Perry <ross.perry@Rosss-MBP-2.home> Co-authored-by: Ross Perry <ross.perry@Rosss-MacBook-Pro-2.local> Co-authored-by: Alex Swindler <alex.swindler@nrel.gov>
* descendant tree endpoint * refactor group merges --------- Co-authored-by: Ross Perry <ross.perry@Rosss-MacBook-Pro-2.local>
* Init systems and services * Add Create Service UI and API * create factories for groups and systems * system test * systems uigrids functional documentation * partial for system table * Add connection type to meters * refactor modal * building update endpoint * system endpoints * service crud functional precommit * service sad path tests error catching service tests * system sad path tests * front end error handling * Add meter import to sytems (#4857) * Fix rollup table (#4835) * default to float data type for axis data if data_type is not set on the selected column --------- Co-authored-by: kflemin <2205659+kflemin@users.noreply.github.com> * More cts fixes (#4830) Co-authored-by: Katherine Fleming <2205659+kflemin@users.noreply.github.com> * Init system meter upload * Lock sass dependency and enforce Node v20 (#4855) * use node version 20 for unittests --------- Co-authored-by: Caleb Rutan <crutan@users.noreply.github.com> Co-authored-by: kflemin <2205659+kflemin@users.noreply.github.com> Co-authored-by: Alex Swindler <Alex.Swindler@nrel.gov> Co-authored-by: Ross Perry <ross.perry@Rosss-MacBook-Pro-2.local> Co-authored-by: Ross Perry <perryr16@gmail.com> * fix test conflict --------- Co-authored-by: Hannah Eslinger <heslinge@nrel.gov> Co-authored-by: Ross Perry <ross.perry@Rosss-MacBook-Pro-2.local> Co-authored-by: Caleb Rutan <crutan@users.noreply.github.com> Co-authored-by: kflemin <2205659+kflemin@users.noreply.github.com> Co-authored-by: Alex Swindler <Alex.Swindler@nrel.gov>
* Add modal to update meter connections * Create update system meter connection endpoint * modal development * hookup group meter update * Fix tests * add config to serailized meter * precommit * comments * fix test * Add dashbard endpoint * group header styling * readable keys * sort options precommit precommit * migration * precommit * fix test * hookup group meter delete --------- Co-authored-by: Hannah Eslinger <heslinge@nrel.gov> Co-authored-by: Ross Perry <ross.perry@Rosss-MacBook-Pro-2.local>
* systen service modal mods * update meter enums * property display field * alphabetical actions * nav error and modal arrangement * precommit * unit refactors * precommit * fix tests --------- Co-authored-by: Ross Perry <ross.perry@Rosss-MacBook-Pro-2.local>
* Make create group meters modal * Remove add meter button from systems and services page * Init Group meter readings upload * Fix dupe error * Fix small things * Make little UI fixes * Add import/export to dashboard * Remove meter attched to other services from groups meter/meterreading list * Format * Format * Format * linter * translations and cleanup * lint * precommit * dismiss action * Fix meter usage * Add message to mismatched meter readings * Fix docker * styles & linter styles & linter * fix group meter usage test --------- Co-authored-by: Ross Perry <ross.perry@Rosss-MacBook-Pro-2.local> Co-authored-by: kflemin <2205659+kflemin@users.noreply.github.com>
haneslinger
added
the
Feature
Add this label to new features. This will be reflected in the change log when generated.
label
Dec 11, 2024
…atform/seed into inventory-groups-develop
…atform/seed into inventory-groups-develop
lint
…atform/seed into inventory-groups-develop
kflemin
approved these changes
Dec 13, 2024
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.
🏆
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Feature
Add this label to new features. This will be reflected in the change log when generated.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#4849