Skip to content

chore: add first kit blog resource#10

Merged
ECWireless merged 1 commit into
mainfrom
chore/add-first-kit-blog-resource
Jun 3, 2026
Merged

chore: add first kit blog resource#10
ECWireless merged 1 commit into
mainfrom
chore/add-first-kit-blog-resource

Conversation

@ECWireless
Copy link
Copy Markdown
Member

@ECWireless ECWireless commented Jun 3, 2026

This pull request updates the marketplace kit page layout and improves the marketplace item data. The main changes enhance the visual presentation of kit images and update the build log description with a published link.

Marketplace kit page layout improvements:

  • Refactored the image section to appear inside the main kit details card, adjusted minimum heights for different screen sizes, and updated the sizes attribute for better responsive image loading in page.tsx. (src/app/marketplace/[slug]/page.tsxL85-R100)

Marketplace item data updates:

  • Updated the build log entry in marketplace.ts to include a published walkthrough link and a more descriptive summary, replacing the "Coming soon" placeholder.

Summary by CodeRabbit

  • Updates

    • Marketplace resources now display published content instead of placeholders, including available walkthrough guides with direct links.
  • Style

    • Marketplace kit pages feature optimized hero image layout with improved responsive design for better visual presentation across devices.

Copilot AI review requested due to automatic review settings June 3, 2026 16:49
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
raidguild-forge Ready Ready Preview, Comment Jun 3, 2026 4:49pm

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f77bcdb5-9165-450c-93ad-dcaa802d35f7

📥 Commits

Reviewing files that changed from the base of the PR and between 5895149 and 41230d8.

📒 Files selected for processing (2)
  • src/app/marketplace/[slug]/page.tsx
  • src/lib/marketplace.ts

📝 Walkthrough

Walkthrough

The marketplace kit detail page is restructured to reposition the hero image from a standalone section into the main content column, improving layout coherence. Simultaneously, the "Build log" marketplace resource transitions from placeholder to published content with description and link.

Changes

Marketplace Kit Detail Update

Layer / File(s) Summary
Hero image layout restructuring
src/app/marketplace/[slug]/page.tsx
Image wrapper moved into main content column beneath kit summary with updated border/background/shadow styling and responsive next/image sizing (sizes="(min-width: 1024px) 58vw, 100vw"), removing the previous standalone section.
Build log resource publication
src/lib/marketplace.ts
"Build log" resource entry replaces placeholder "coming soon" description with published walkthrough text and adds concrete href link.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • raid-guild/raidguild-forge#7: Implements the initial marketplace kit detail page and resource structure that this PR refines with layout reorganization and content publication.

Poem

🐰 A rabbit hops through the layout so neat,
Images dancing to find their sweet seat—
From bottom to top, they gracefully flow,
While build logs now publish their knowledge to show! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: add first kit blog resource' describes adding a blog resource, which matches the marketplace item data update that replaces a placeholder with a published walkthrough description and link. However, the PR also includes significant layout refactoring changes to the marketplace kit page that are not reflected in the title.
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.

✏️ 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 chore/add-first-kit-blog-resource

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed: lockfile failed supply-chain policy check. Run pnpm install locally to update the lockfile.


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

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

This pull request improves the marketplace kit detail experience by moving the kit image into the primary details card layout and by updating the kit’s “Build log” resource to link to the now-published walkthrough.

Changes:

  • Refactored the marketplace kit detail layout so the hero image renders inside the main kit details card with updated responsive min-heights and sizes.
  • Updated the marketplace item “Build log” resource to replace the placeholder status/description with a published Paragraph link and summary.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/lib/marketplace.ts Updates the kit “Build log” resource entry to include a published walkthrough link and improved description.
src/app/marketplace/[slug]/page.tsx Moves and adjusts the kit image presentation within the main details section and refines responsive image sizing.

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

@ECWireless ECWireless merged commit 1c727f0 into main Jun 3, 2026
4 checks passed
@ECWireless ECWireless deleted the chore/add-first-kit-blog-resource branch June 3, 2026 16:51
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