Feature/ab#29055 reporting configuration #1746
Merged
Conversation
This file contains hidden or 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
…ting-configuration
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
....Reporting/src/Unity.Reporting.Web/Views/Shared/Components/ReportingConfiguration/Default.js
Dismissed
Show dismissed
Hide dismissed
....Reporting/src/Unity.Reporting.Web/Views/Shared/Components/ReportingConfiguration/Default.js
Dismissed
Show dismissed
Hide dismissed
...odules/Unity.Reporting/src/Unity.Reporting.Application/Configuration/ReportMappingService.cs
Dismissed
Show dismissed
Hide dismissed
...c/Unity.Reporting.Application/EntityFrameworkCore/Repositories/ReportColumnsMapRepository.cs
Dismissed
Show dismissed
Hide dismissed
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request introduces a comprehensive Reporting Configuration Feature that enables administrators to configure column names for report views and explicitly generate database views for form submissions, worksheets, and scoresheets with role-based access control.
Key Changes:
- Added reporting configuration infrastructure with database schema for column mappings and view management
- Implemented PostgreSQL stored procedures and functions for dynamic view generation from JSON data
- Created a form metadata service to analyze and extract component information from form schemas
- Added UI components for reporting configuration with status tracking and preview capabilities
Reviewed Changes
Copilot reviewed 149 out of 159 changed files in this pull request and generated 23 comments.
Show a summary per file
| File | Description |
|---|---|
| FormMetadataServiceTests.cs | Comprehensive test suite for form metadata extraction and component analysis |
| Mapping.cshtml | Added reporting configuration tab to application forms mapping page |
| PolicyRegistrant.cs | Registered reporting configuration authorization policies |
| Unity.GrantManager.EntityFrameworkCore.csproj | Embedded SQL scripts as resources for database migrations |
| get_worksheet_data.sql | PostgreSQL function for extracting worksheet data into structured views |
| get_scoresheet_data.sql | PostgreSQL function for extracting scoresheet assessment data |
| get_formversion_data.sql | PostgreSQL function for extracting form submission data |
| generate_*_view.sql | PostgreSQL stored procedures for creating reporting views |
| calculate_scoresheet_total_score.sql | Function for calculating scoresheet total scores |
| README.md | Documentation for database scripts and migration dependencies |
| Multiple migrations | Database schema changes for reporting configuration tables |
| FormMetadataService.cs | Service implementation for analyzing form schemas and extracting metadata |
| FormsApiService.cs | Removed unused import statement |
| ReportingConfigurationViewStatus components | UI components for displaying view generation status with preview |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...ng/src/Unity.Reporting.Application/Configuration/FieldsProviders/ScoresheetFieldsProvider.cs
Show resolved
Hide resolved
.../modules/Unity.Reporting/src/Unity.Reporting.Application/Configuration/ReportMappingUtils.cs
Show resolved
Hide resolved
.../modules/Unity.Reporting/src/Unity.Reporting.Application/Configuration/ReportMappingUtils.cs
Show resolved
Hide resolved
...antManager/src/Unity.GrantManager.Application/Reporting/Configuration/FormMetadataService.cs
Show resolved
Hide resolved
...antManager/src/Unity.GrantManager.Application/Reporting/Configuration/FormMetadataService.cs
Show resolved
Hide resolved
...ed/Components/ReportingConfigurationViewStatus/ReportingConfigurationViewStatusController.cs
Show resolved
Hide resolved
...Components/ReportingConfigurationViewStatus/ReportingConfigurationViewStatusViewComponent.cs
Show resolved
Hide resolved
...antManager/src/Unity.GrantManager.Application/Reporting/Configuration/FormMetadataService.cs
Outdated
Show resolved
Hide resolved
...ng.Web/Views/Shared/Components/ReportingConfiguration/ReportingConfigurationViewComponent.cs
Show resolved
Hide resolved
...rting.Web/Views/Shared/Components/ReportingConfiguration/ReportingConfigurationController.cs
Show resolved
Hide resolved
JamesPasta
approved these changes
Nov 18, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Pull Request Overview
This pull request introduces a comprehensive Reporting Configuration Feature that enables administrators to configure column names for report views and explicitly generate database views for form submissions, worksheets, and scoresheets with role-based access control.
Key Changes: