Skip to content

Issue 2055: Add warning for report year selection in facility analysis setup#2102

Merged
rmroot merged 1 commit intodevelopfrom
issue-2055
Oct 15, 2025
Merged

Issue 2055: Add warning for report year selection in facility analysis setup#2102
rmroot merged 1 commit intodevelopfrom
issue-2055

Conversation

@rmroot
Copy link
Copy Markdown
Member

@rmroot rmroot commented Oct 15, 2025

connects #2055

This pull request adds a warning and supporting logic to notify users when a facility lacks a full year of data for the selected report year in the analysis setup. The goal is to guide users to use the "Savings Report" feature for mid-year progress instead of running an analysis with incomplete data.

User interface improvements:

  • Added a warning message to analysis-setup.component.html that displays when the selected report year does not have 12 months of data, including a link to the "Savings Report" for mid-year progress.

Component logic and behavior:

  • Introduced the showReportYearWarning property and implemented the setReportYearWarning method in analysis-setup.component.ts to determine when to show the warning based on data completeness for the selected report year. [1] [2] [3]
  • Updated the changeReportYear method to invoke setReportYearWarning whenever the report year changes, ensuring the warning is shown or hidden appropriately.
  • Added the goToSavingsReport method to navigate users to the "Savings Report" page when the link in the warning is clicked.

Code maintenance:

  • Updated imports in analysis-setup.component.ts to include ActivatedRoute for improved navigation handling.
  • Minor formatting fix in the template for better readability.

@rmroot rmroot merged commit c963442 into develop Oct 15, 2025
3 checks passed
@rmroot rmroot deleted the issue-2055 branch November 5, 2025 16:47
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.

1 participant