Skip to content

fix category blog field#1576

Open
aline-pereira wants to merge 1 commit intomainfrom
fix/blog-fields
Open

fix category blog field#1576
aline-pereira wants to merge 1 commit intomainfrom
fix/blog-fields

Conversation

@aline-pereira
Copy link
Copy Markdown
Contributor

@aline-pereira aline-pereira commented Apr 15, 2026

What is this Contribution About?

Please provide a brief description of the changes or enhancements you are proposing in this pull request.

Issue Link

Please link to the relevant issue that this pull request addresses:

Loom Video

Record a quick screencast describing your changes to help the team understand and review your contribution. This will greatly assist in the review process.

Demonstration Link

Provide a link to a branch or environment where this pull request can be tested and seen in action.

Summary by CodeRabbit

  • Documentation
    • Updated type documentation annotations to improve metadata handling and clarity for category definitions.

Summary by cubic

Fixed the blog post category field so categories display and can be selected correctly in the editor. Added @titleby name to Category and removed the invalid widget/collection annotations from BlogPost.categories.

Written for commit 5e1f243. Summary will update on new commits.

@github-actions
Copy link
Copy Markdown
Contributor

Tagging Options

Should a new tag be published when this PR is merged?

  • 👍 for Patch 0.145.1 update
  • 🎉 for Minor 0.146.0 update
  • 🚀 for Major 1.0.0 update

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 15, 2026

📝 Walkthrough

Walkthrough

Updated JSDoc annotations in the blog types module, adding a @titleBy name marker to the Category interface and removing JSDoc metadata from the BlogPost.categories property. No functional type changes were made.

Changes

Cohort / File(s) Summary
Blog Types Documentation
blog/types.ts
Added @titleBy name JSDoc annotation to Category interface; removed @widget blog and @collection categories JSDoc tags from BlogPost.categories property.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • guitavano

Poem

🐰 A hop through the docs so neat,
Where categories now have their meet,
With @titleBy name we declare,
Blog metadata organized with care!
Metadata trimmed, annotations refined,

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'fix category blog field' is vague and does not clearly describe the actual changes made to the codebase. Provide a more specific title that describes the nature of the fix, such as 'Add @titleBy JSDoc annotation to Category interface and remove blog metadata tags' to help reviewers understand the exact changes.
Description check ❓ Inconclusive The pull request description uses the required template structure but contains mostly empty sections with placeholder text, though an auto-generated summary provides technical context about the changes. The author should complete the 'What is this Contribution About?' section with a clear description of the changes, provide the Issue Link, and optionally add Loom Video and Demonstration Link to fully meet contribution guidelines.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/blog-fields

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

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.

1 participant