Skip to content

refactor(blog): remove imageCarousel from BlogPost - #1684

Merged
aka-sacci-ccr merged 1 commit into
mainfrom
delete-image-carousel-blog
Oct 7, 2026
Merged

aka-sacci-ccr merged 1 commit into
mainfrom
delete-image-carousel-blog

Conversation

@aka-sacci-ccr

@aka-sacci-ccr aka-sacci-ccr commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

What

Removes the imageCarousel field from BlogPost and the ImageCarousel interface in blog/types.ts.

Why

The field was never implemented — its own JSDoc said "Must be implemented in frontEnd" — and nothing in this repo reads it. Dropping it removes a dead knob from the blog post form in the CMS.

Notes

Banner / BannerItem are now unreferenced inside this repo, but were left in place since they're exported types that downstream sites may import. Say the word and I'll remove those too.

🤖 Generated with Claude Code


Summary by cubic

Removes the unused imageCarousel field from BlogPost and the ImageCarousel interface in blog/types.ts. The field was never implemented in the frontend and nothing in this repo reads it, so it no longer shows up as a dead option in the CMS blog post form.

Banner and BannerItem are left in place even though nothing in this repo references them anymore, since they're exported types that downstream sites may import.

Written for commit 7afece2. Summary will update on new commits.

View guided diff Turn on auto-fix

Summary by CodeRabbit

  • Updates
    • Blog posts no longer support the image carousel property.

Drop the unimplemented imageCarousel field and its ImageCarousel
interface. Nothing in the repo consumed either one.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Tagging Options

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

  • 👍 for Patch 0.165.2 update
  • 🎉 for Minor 0.166.0 update
  • 🚀 for Major 1.0.0 update

@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b26dd7e2-5864-48b5-8cde-c06070dbb3cd
📥 Commits

Reviewing files that changed from the base of the PR and between 7147feb and 7afece2.

📒 Files selected for processing (1)
  • blog/types.ts
💤 Files with no reviewable changes (1)
  • blog/types.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The BlogPost type no longer declares an imageCarousel property. The exported ImageCarousel interface was removed.

Changes

Blog type declarations

Layer / File(s) Summary
Remove carousel declarations
blog/types.ts
Removed BlogPost.imageCarousel and the exported ImageCarousel interface, including its optional banners and description fields.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Refactor

Merge Risk: ⚪ Minimal · up to 7afec

The unused carousel declarations have been removed; no actionable merge risk is identified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removing imageCarousel from BlogPost.
Description check ✅ Passed The description clearly explains what changed and why, and includes relevant implementation notes. It does not include an issue link, Loom video, or demonstration link, but the core description is com…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

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

Copy link
Copy Markdown

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

View guided diff | Turn on auto-fix | Re-trigger cubic

@aka-sacci-ccr
aka-sacci-ccr merged commit 440f240 into main Oct 7, 2026
7 checks passed
@aka-sacci-ccr
aka-sacci-ccr deleted the delete-image-carousel-blog branch October 7, 2026 19:54
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