Skip to content

fix: add missing description prop to Spinner in FieldsCard#55

Merged
antoniomtz merged 1 commit intomainfrom
antoniomtz/fix-spinner-aria-label
Apr 17, 2026
Merged

fix: add missing description prop to Spinner in FieldsCard#55
antoniomtz merged 1 commit intomainfrom
antoniomtz/fix-spinner-aria-label

Conversation

@antoniomtz
Copy link
Copy Markdown
Collaborator

@antoniomtz antoniomtz commented Apr 17, 2026 •
edited
Loading

Summary

  • Adds the required description prop to the Spinner component in FieldsCard.tsx, fixing a TypeScript build error that broke the UI Docker container build
  • The description prop is required by the KUI Spinner component for accessibility (aria-label or description)
  • This regression was introduced in PR feat: product manual PDF upload for enhanced FAQ generation #53

Test plan

  • Verified pnpm run build completes successfully
  • Rebuild UI Docker container and confirm it builds without errors

🤖 Generated with Claude Code

@antoniomtz @claude
fix: add missing description prop to Spinner in FieldsCard
de93b28 
The Spinner component requires a `description` or `aria-label` prop for
accessibility. The missing prop caused a TypeScript build failure in the
UI Docker container.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@antoniomtz antoniomtz self-assigned this Apr 17, 2026
@antoniomtz antoniomtz added the bug Something isn't working label Apr 17, 2026
@antoniomtz antoniomtz merged commit 562f6d5 into main Apr 17, 2026
3 of 4 checks passed
@antoniomtz antoniomtz deleted the antoniomtz/fix-spinner-aria-label branch April 17, 2026 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@antoniomtz antoniomtz

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@antoniomtz