feat: regroup assets into folders by houssembaltii · Pull Request #625 · Fork-It-Community/forkit.community

houssembaltii · 2026-02-10T13:54:06Z

Summary by CodeRabbit

Refactor
- Internal utility references adjusted across event asset components for consistent module resolution.
- Event assets reorganized into dedicated subdirectories (announcements, banners, call-for-papers, QR codes, save-the-date, schedule, tickets) for clearer structure.
New Features
- Asset loading now discovers files in deeper nested folders, allowing additional nested assets to be registered and served.
- Events assets route expanded to expose the new asset categories under each event.

vercel · 2026-02-10T13:54:13Z

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

Project	Deployment	Actions	Updated (UTC)
forkit-community	Ready	Preview, Comment	Apr 10, 2026 11:21am

coderabbitai · 2026-02-10T13:54:26Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e1e3553f-b354-4108-a613-933c8996317b

📥 Commits

Reviewing files that changed from the base of the PR and between 52688f3 and 3a86e2c.

📒 Files selected for processing (29)

src/pages/events/[id]/assets/[__image].[__type].ts
src/pages/events/[id]/assets/announcements/_d-1-announcement-insta.tsx
src/pages/events/[id]/assets/announcements/_d-1-announcement.tsx
src/pages/events/[id]/assets/announcements/_d-10-announcement-insta.tsx
src/pages/events/[id]/assets/announcements/_d-10-announcement.tsx
src/pages/events/[id]/assets/announcements/_d-30-announcement-insta.tsx
src/pages/events/[id]/assets/announcements/_d-30-announcement.tsx
src/pages/events/[id]/assets/announcements/_d-7-announcement-insta.tsx
src/pages/events/[id]/assets/announcements/_d-7-announcement.tsx
src/pages/events/[id]/assets/banners/_banner-linkedin.tsx
src/pages/events/[id]/assets/banners/_banner-youtube.tsx
src/pages/events/[id]/assets/cfp/_cfp-cover.tsx
src/pages/events/[id]/assets/cfp/_cfp-open-insta.tsx
src/pages/events/[id]/assets/cfp/_cfp-open.tsx
src/pages/events/[id]/assets/cfp/_cfp-still-open-insta.tsx
src/pages/events/[id]/assets/cfp/_cfp-still-open.tsx
src/pages/events/[id]/assets/qrcode/_qrcode-bg.tsx
src/pages/events/[id]/assets/qrcode/_qrcode.tsx
src/pages/events/[id]/assets/save-the-date/_save-the-date-insta.tsx
src/pages/events/[id]/assets/save-the-date/_save-the-date-square.tsx
src/pages/events/[id]/assets/save-the-date/_save-the-date.tsx
src/pages/events/[id]/assets/schedule/_schedule-short-meetup.tsx
src/pages/events/[id]/assets/tickets/_early-bird-tickets-insta.tsx
src/pages/events/[id]/assets/tickets/_early-bird-tickets.tsx
src/pages/events/[id]/assets/tickets/_registration-still-open-insta.tsx
src/pages/events/[id]/assets/tickets/_registration-still-open.tsx
src/pages/events/[id]/assets/tickets/_tickets-available-insta.tsx
src/pages/events/[id]/assets/tickets/_tickets-available.tsx
src/routes.gen.ts

✅ Files skipped from review due to trivial changes (20)

src/pages/events/[id]/assets/announcements/_d-1-announcement.tsx
src/pages/events/[id]/assets/announcements/_d-1-announcement-insta.tsx
src/pages/events/[id]/assets/qrcode/_qrcode.tsx
src/pages/events/[id]/assets/cfp/_cfp-still-open.tsx
src/pages/events/[id]/assets/cfp/_cfp-cover.tsx
src/pages/events/[id]/assets/tickets/_tickets-available.tsx
src/pages/events/[id]/assets/tickets/_tickets-available-insta.tsx
src/pages/events/[id]/assets/announcements/_d-10-announcement-insta.tsx
src/pages/events/[id]/assets/announcements/_d-10-announcement.tsx
src/pages/events/[id]/assets/save-the-date/_save-the-date-insta.tsx
src/pages/events/[id]/assets/cfp/_cfp-still-open-insta.tsx
src/pages/events/[id]/assets/qrcode/_qrcode-bg.tsx
src/pages/events/[id]/assets/banners/_banner-linkedin.tsx
src/pages/events/[id]/assets/schedule/_schedule-short-meetup.tsx
src/pages/events/[id]/assets/announcements/_d-30-announcement.tsx
src/pages/events/[id]/assets/banners/_banner-youtube.tsx
src/pages/events/[id]/assets/announcements/_d-7-announcement.tsx
src/pages/events/[id]/assets/tickets/_early-bird-tickets.tsx
src/pages/events/[id]/assets/save-the-date/_save-the-date.tsx
src/pages/events/[id]/assets/announcements/_d-30-announcement-insta.tsx

🚧 Files skipped from review as they are similar to previous changes (5)

src/pages/events/[id]/assets/cfp/_cfp-open-insta.tsx
src/pages/events/[id]/assets/cfp/_cfp-open.tsx
src/pages/events/[id]/assets/[__image].[__type].ts
src/routes.gen.ts
src/pages/events/[id]/assets/announcements/_d-7-announcement-insta.tsx

📝 Walkthrough

Walkthrough

Broadened the asset file glob to recursively include nested asset components, reorganized event assets into subdirectories, updated many getEventData import paths to the parent _utils, and added corresponding nested events/:id/assets route entries in the generated routes.

Changes

Cohort / File(s)	Summary
Asset Endpoint Glob Pattern `src/pages/events/[id]/assets/[__image].[__type].ts`	Broadened import glob from `./_.tsx` to `.//_.tsx` to eagerly include nested `_*.tsx` asset modules.
Announcements `src/pages/events/[id]/assets/announcements/_d-1-announcement.tsx`, `src/pages/events/[id]/assets/announcements/_d-1-announcement-insta.tsx`, `src/pages/events/[id]/assets/announcements/_d-7-announcement.tsx`, `src/pages/events/[id]/assets/announcements/_d-7-announcement-insta.tsx`, `src/pages/events/[id]/assets/announcements/_d-10-announcement.tsx`, `src/pages/events/[id]/assets/announcements/_d-10-announcement-insta.tsx`, `src/pages/events/[id]/assets/announcements/_d-30-announcement.tsx`, `src/pages/events/[id]/assets/announcements/_d-30-announcement-insta.tsx`	Updated `getEventData` relative imports from `./_utils` → `../_utils`.
Banners `src/pages/events/[id]/assets/banners/_banner-linkedin.tsx`, `src/pages/events/[id]/assets/banners/_banner-youtube.tsx`	Updated `getEventData` import path from `./_utils` → `../_utils`.
CFP `src/pages/events/[id]/assets/cfp/_cfp-open.tsx`, `src/pages/events/[id]/assets/cfp/_cfp-open-insta.tsx`, `src/pages/events/[id]/assets/cfp/_cfp-still-open.tsx`, `src/pages/events/[id]/assets/cfp/_cfp-still-open-insta.tsx`, `src/pages/events/[id]/assets/cfp/_cfp-cover.tsx`	Moved CFP assets into `cfp/`; updated `getEventData` imports to `../_utils` and adjusted paths to match nested layout.
QR Code `src/pages/events/[id]/assets/qrcode/_qrcode.tsx`, `src/pages/events/[id]/assets/qrcode/_qrcode-bg.tsx`	Updated `getEventData` import path from `./_utils` → `../_utils`.
Save-the-Date `src/pages/events/[id]/assets/save-the-date/_save-the-date.tsx`, `src/pages/events/[id]/assets/save-the-date/_save-the-date-insta.tsx`	Updated `getEventData` import path from `./_utils` → `../_utils`.
Schedule `src/pages/events/[id]/assets/schedule/_schedule-short-meetup.tsx`	Updated `getEventData` import `./_utils` → `../_utils`; adjusted `getTalkData` path from `../talks/[talkId]/assets/_utils` → `../../talks/[talkId]/assets/_utils`.
Tickets `src/pages/events/[id]/assets/tickets/_tickets-available.tsx`, `src/pages/events/[id]/assets/tickets/_tickets-available-insta.tsx`, `src/pages/events/[id]/assets/tickets/_early-bird-tickets.tsx`, `src/pages/events/[id]/assets/tickets/_early-bird-tickets-insta.tsx`, `src/pages/events/[id]/assets/tickets/_registration-still-open.tsx`, `src/pages/events/[id]/assets/tickets/_registration-still-open-insta.tsx`	Updated `getEventData` import path from `./_utils` → `../_utils`.
Route Configuration `src/routes.gen.ts`	Added nested `assets` subroutes under `events[":id"]`: `announcements`, `banners`, `cfp`, `qrcode`, `save-the-date`, `schedule`, and `tickets` (retaining existing `download`).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

update routes.gen.ts #613 — overlaps changes to src/routes.gen.ts asset route definitions.
feat: organize page assets #588 — reorganizes event assets and the _utils used by asset components.
Asset: cfp open #493 — adds CFP asset files that overlap the CFP assets moved/updated here.

Suggested reviewers

yoannfleurydev
JeanneGrenet

Poem

🐰 I hopped through nested folders, tidy and spry,
I widened the globs so small assets don't hide,
I nudged imports upward, one dot at a time,
Now event cards gather — organized rhyme! 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: reorganizing assets into folders by moving import paths from local modules to parent directories and updating route configuration.
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 feat/regroup-assets-into-folders

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.}

houssembaltii · 2026-02-12T09:49:03Z

imrpove when this is merged : #588

Clean up leftover rebase conflicts: remove duplicated files at the assets root, resolve committed conflict markers in cfp insta files, restore the simplified delegation form for save-the-date and tickets-available insta, and move early-bird-tickets and registration-still-open into tickets/. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Revert accidental content changes to the four insta asset files (cfp-open, cfp-still-open, save-the-date, tickets-available) that were refactored into thin wrappers during the regroup. Restores the full original implementations from main, keeping only the folder move and the `_utils` import path update. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

houssembaltii self-assigned this Feb 10, 2026

houssembaltii force-pushed the feat/regroup-assets-into-folders branch from 5717fc2 to 52688f3 Compare February 12, 2026 09:22

vercel Bot deployed to Preview February 12, 2026 09:24 View deployment

houssembaltii marked this pull request as draft February 12, 2026 09:49

houssembaltii and others added 2 commits April 10, 2026 10:29

feat: regroup assets into folders

0ef40c5

houssembaltii force-pushed the feat/regroup-assets-into-folders branch from 52688f3 to 5ddfae2 Compare April 10, 2026 09:56

vercel Bot deployed to Preview April 10, 2026 09:57 View deployment

vercel Bot deployed to Preview April 10, 2026 11:21 View deployment

houssembaltii marked this pull request as ready for review April 10, 2026 15:24

houssembaltii requested review from JeanneGrenet and yoannfleurydev April 10, 2026 15:24

houssembaltii marked this pull request as draft April 14, 2026 12:42

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: regroup assets into folders#625

feat: regroup assets into folders#625
houssembaltii wants to merge 3 commits into
mainfrom
feat/regroup-assets-into-folders

houssembaltii commented Feb 10, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

vercel Bot commented Feb 10, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Feb 10, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Uh oh!

houssembaltii commented Feb 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

houssembaltii commented Feb 10, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

vercel Bot commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Uh oh!

houssembaltii commented Feb 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

houssembaltii commented Feb 10, 2026 •

edited by coderabbitai Bot

Loading

vercel Bot commented Feb 10, 2026 •

edited

Loading

coderabbitai Bot commented Feb 10, 2026 •

edited

Loading