Skip to content

feat: add description field to data.media#94

Open
dannda wants to merge 1 commit intodevfrom
feat/data-media-description
Open

feat: add description field to data.media#94
dannda wants to merge 1 commit intodevfrom
feat/data-media-description

Conversation

@dannda
Copy link
Contributor

@dannda dannda commented Mar 2, 2026

Description

add description field to data.media to hold text longer that the media caption

fixes #93

Type of change

  • 🐛 Bug fix (non-breaking change that resolves an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • ⚡ Optimisation (non-breaking improvement to performance or efficiency)
  • 🧩 Documentation (adds or improves documentation)
  • 🧱 Maintenance (refactor, dependency update, CI/CD, etc.)
  • 🔥 Breaking change (fix or feature that causes existing functionality to change)

Checklist

  • All tests pass (eg. npm test)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)
  • Documentation updated (if required)

Copilot AI review requested due to automatic review settings March 2, 2026 10:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new long-form text field to the data.media Strapi component so media entries can store additional text beyond the underlying uploaded asset’s caption limit (per #93).

Changes:

  • Add description attribute (type text) to src/components/data/media.json.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"title": {
"type": "string"
},
"description": {
Copy link

Copilot AI Mar 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Issue #93 and the component’s semantics describe this as a (long) "caption" for the media, but the new attribute is named "description". Since this becomes part of the public API, consider renaming the field to "caption" (or another clearly caption-specific name), or update the issue/consumer expectations to avoid ambiguity with other "description" fields in the schema.

Suggested change
"description": {
"caption": {

Copilot uses AI. Check for mistakes.
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.

2 participants