Skip to content

4558 reporting field display titles - #4661

Open
gdalcengio wants to merge 2 commits into
developfrom
4558-reporting-field-display-titles
Open

4558 reporting field display titles#4661
gdalcengio wants to merge 2 commits into
developfrom
4558-reporting-field-display-titles

Conversation

@gdalcengio

@gdalcengio gdalcengio commented May 13, 2026

Copy link
Copy Markdown
Contributor

resolves: 4558

  • added reporting fields map excluding the null fields to facilityReport through a resolver
  • passed it through the recursive renderObject function in order to check at the field level

Also fixes the review-changes field rendering

  • deleted /bciers/apps/reporting/src/app/components/changeReview/components/ObjectRenderer.tsx as it wasn't being used anywhere
  • added reporting fields map excluding the null fields to changes service
  • passed it all the way through the changes recursion diff trees

@gdalcengio
gdalcengio force-pushed the 4558-reporting-field-display-titles branch 4 times, most recently from fc34a51 to cfaa60d Compare May 14, 2026 23:46
@gdalcengio
gdalcengio force-pushed the 4558-reporting-field-display-titles branch from cfaa60d to f8e5122 Compare May 15, 2026 00:38
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.

Reporting/Activity labels in Final Review & Review Changes pages

1 participant