Skip to content

Issue 2231 - added option for user to choose whether to export weather predictors#2245

Merged
rmroot merged 2 commits intodevelopfrom
issue-2231
Feb 25, 2026
Merged

Issue 2231 - added option for user to choose whether to export weather predictors#2245
rmroot merged 2 commits intodevelopfrom
issue-2231

Conversation

@jamlokim
Copy link
Copy Markdown
Contributor

connects #2231

This pull request introduces a new modal dialog for exporting account and facility data, allowing users to optionally include weather data in their exports. The changes affect the Manage Accounts, Account Home Summary, and Facility Home Summary components, updating both their UI and logic to support this new feature. Additionally, some minor UI and formatting improvements have been made.

Export Modal and Weather Data Option:

  • Added an export modal dialog to ManageAccountsComponent, AccountHomeSummaryComponent, and FacilityHomeSummaryComponent, allowing users to choose whether to include weather data when exporting to Excel. The modal is triggered by a new "Export" button and includes a styled checkbox for the weather data option.

  • Updated export logic in the relevant components and services to pass the includeWeatherData flag, ensuring the export includes weather data if selected.

UI/Styling Improvements:

  • Added .checkbox-style CSS classes for consistent checkbox sizing and label styling in export modals across components.

Dependency Updates:

  • Imported FormsModule in AccountHomeModule and FacilityHomeModule to support two-way binding for the weather data checkbox in the export modals.
    Minor Formatting and Clean-up:

  • Cleaned up HTML formatting and improved consistency in button and badge layouts in several components.

These changes collectively enhance the export functionality by making it more flexible and user-friendly, while also improving code maintainability and UI consistency.

@jamlokim jamlokim closed this Feb 11, 2026
@jamlokim jamlokim reopened this Feb 11, 2026
@jamlokim jamlokim requested a review from rmroot February 11, 2026 20:47
@rmroot rmroot merged commit 675caac into develop Feb 25, 2026
3 checks passed
@jamlokim jamlokim deleted the issue-2231 branch March 25, 2026 20:56
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