Skip to content

Add Upload component for project proposals#81

Open
BIA3IA wants to merge 5 commits into
mainfrom
bianca/upload
Open

Add Upload component for project proposals#81
BIA3IA wants to merge 5 commits into
mainfrom
bianca/upload

Conversation

@BIA3IA
Copy link
Copy Markdown
Contributor

@BIA3IA BIA3IA commented Apr 14, 2026

Introduce an Upload component to propose new projects or suggest improvements.

Closes #76

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 14, 2026

Review Change Stack
No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ea30901d-5996-4081-be4d-be12253e892e

📥 Commits

Reviewing files that changed from the base of the PR and between f34cf16 and 6ff1edd.

📒 Files selected for processing (3)
  • src/app/projects/page.tsx
  • src/components/card-caption.tsx
  • src/components/projects/upload.tsx

Walkthrough

The PR adds a new Projects page with an Upload component displaying contact and requirements information. CardCaption is updated to accept ReactNode captions instead of strings and forwards an optional className prop to its Card root, enabling the new component to render complex JSX content within cards.

Changes

Projects Upload Feature

Layer / File(s) Summary
CardCaption contract expansion
src/components/card-caption.tsx
CardCaption prop contract changes from caption: string to caption: ReactNode, conditionally wrapping string captions in <p> and rendering non-string nodes directly. Optional className prop is added and forwarded to the root Card element.
Upload component with CardCaption content
src/components/projects/upload.tsx
Upload component defines a local cards configuration with two sections: "Contatti" (contact details with mail and social icons) and "Requisiti" (requirements as a bulleted list). The component renders a full-page section with headline, descriptive text, and CardCaption instances mapped from cards.
Projects page route
src/app/projects/page.tsx
New Projects page at the app root imports and renders the Upload component inside a main container with full-width styling.

Possibly Related PRs

  • PoliNetworkOrg/web#41: Both PRs modify src/components/card-caption.tsx—the related PR originally added CardCaption and this PR extends its API to accept ReactNode captions and className forwarding.
🚥 Pre-merge checks | ✅ 3 | ❌ 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 (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add Upload component for project proposals' clearly and specifically describes the main change in the changeset.
Linked Issues check ✅ Passed The PR implements the main coding requirement from issue #76 by adding the Upload component for project proposals with proper UI structure.
Out of Scope Changes check ✅ Passed Changes include the Upload component (in-scope), CardCaption component enhancement to support the Upload component (in-scope), and the Projects page setup (in-scope).

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

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.

@BIA3IA BIA3IA self-assigned this Apr 15, 2026
@BIA3IA BIA3IA marked this pull request as ready for review May 12, 2026 11: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.

Projects - Upload

1 participant