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

Protocol questions for End Uses and System Inventory #401

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

Conversation

rmroot
Copy link
Collaborator

@rmroot rmroot commented Feb 24, 2025

connects #351
connects #352
connects #374
This is a first pass at incorporating protocol questions. The initial vision didn't work so we are compromising at this stage.

  • Adds "Take Stock" protocol questions to the system and end use pages in pre-visit
  • No additional content added to assessment screens
  • Three other categories of questions are shown in the portfolio for systems and end use inventory. These questions will be added more elegantly to assessments in future work. Issues created for this.
  • Field by field help text added for questions in setup wizard. All help shown in forms in portfolio

Minor updates to the end use and system inventory connectivity.
connects #160
connects #206

Other notes:

  • The sidepanel in the setup wizard now has tabs. This came from some initial work on incorporating the protocol in assessment and is something we want to keep. Further issues created to complete the initial work on each new tab
  • Energy details within the System Inventory are now in an accordion to emphasize that they are optional input fields. This is apart of feedback from the team to better identify optional input data show to a user.
  • It was asked to add some color to the sidebar, that was started. Additional issue to fine tune that made

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://gray-tree-06c66911e-401.westus2.4.azurestaticapps.net

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://gray-tree-06c66911e-401.westus2.4.azurestaticapps.net

1 similar comment
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://gray-tree-06c66911e-401.westus2.4.azurestaticapps.net

@rmroot rmroot mentioned this pull request Feb 24, 2025
describeServicingNeeds: string,
describeLaborRequirements: string,
describeSystemMaterials: string;
//sustainability
Copy link
Collaborator

Choose a reason for hiding this comment

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

just a flag that DOE's new guidance on not to use "sustainability" learned from Comms. No need to change anything. Will check with DOE and team about this and create a new issue if we need to do something about it.

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.

2 participants