Skip to content

CS-180 [Bug] Document findings do not link correctly#2289

Merged
Marfuen merged 4 commits intomainfrom
chas/document-findings
Mar 12, 2026
Merged

CS-180 [Bug] Document findings do not link correctly#2289
Marfuen merged 4 commits intomainfrom
chas/document-findings

Conversation

@github-actions
Copy link
Contributor

This is an automated pull request to merge chas/document-findings into dev.
It was created by the [Auto Pull Request] action.

@vercel
Copy link

vercel bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Mar 12, 2026 2:42pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal Skipped Skipped Mar 12, 2026 2:42pm

Request Review

@cursor
Copy link

cursor bot commented Mar 12, 2026

PR Summary

Low Risk
Low risk: small API response normalization plus minor UI label formatting changes; main risk is unexpected form type mapping affecting document routes.

Overview
Fixes document finding navigation by normalizing evidence form types in the API response: getOrganizationFindings now converts finding.evidenceFormType and nested evidenceSubmission.formType via toExternalEvidenceFormType (with a fallback to meeting).

Updates the findings UI to display document types more readably by replacing hyphens with spaces when rendering Document: ... labels.

Written by Cursor Bugbot for commit b31353d. This will update automatically on new commits. Configure here.

@chasprowebdev chasprowebdev changed the title [dev] [chasprowebdev] chas/document-findings CS-180 [Bug] Document findings do not link correctly Mar 12, 2026
@linear
Copy link

linear bot commented Mar 12, 2026

@vercel vercel bot temporarily deployed to Preview – portal March 12, 2026 14:40 Inactive
@Marfuen Marfuen merged commit 2938ee2 into main Mar 12, 2026
7 of 8 checks passed
@Marfuen Marfuen deleted the chas/document-findings branch March 12, 2026 14:40
@claudfuen
Copy link
Contributor

🎉 This PR is included in version 3.2.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants