Skip to content

fix: render envelope validation data correctly for multi-file requests#7647

Merged
vitormattos merged 2 commits intomainfrom
fix/envelope-validation-multifile-display
May 6, 2026
Merged

fix: render envelope validation data correctly for multi-file requests#7647
vitormattos merged 2 commits intomainfrom
fix/envelope-validation-multifile-display

Conversation

@vitormattos
Copy link
Copy Markdown
Member

@vitormattos vitormattos commented May 6, 2026

Summary

  • fix envelope validation UI rendering by importing required @nextcloud/vue components in EnvelopeValidation.vue
  • make documents counter robust by rendering Number of documents when count is 0 and using files.length fallback when filesCount is missing
  • add regression coverage with unit tests and a Playwright E2E reproduction flow for envelope with 2 files

Root Cause

EnvelopeValidation.vue template used Nextcloud components that were not imported in the component script. In this environment that led to unresolved Vue components and broken/partial rendering on envelope validation, while single-file validation kept working.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@github-project-automation github-project-automation Bot moved this to 0. Needs triage in Roadmap May 6, 2026
@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable33

@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable32

@vitormattos vitormattos merged commit d0c59f4 into main May 6, 2026
69 checks passed
@vitormattos vitormattos deleted the fix/envelope-validation-multifile-display branch May 6, 2026 14:17
@github-project-automation github-project-automation Bot moved this from 0. Needs triage to 4. to release in Roadmap May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 4. to release

Development

Successfully merging this pull request may close these issues.

1 participant