feat: add techKoodaram community and Build Day event#531
feat: add techKoodaram community and Build Day event#531MalaiarasuGRaj wants to merge 1 commit intoFOSSUChennai:mainfrom
Conversation
📝 WalkthroughWalkthroughUpdates framer-motion dependency, relocates Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@package.json`:
- Line 34: The package "@phosphor-icons/react" is required at runtime by UI
components (e.g., MapPin in events.tsx and archive.tsx, CalendarX in
no-events-card.tsx, RssSimple in header.tsx, and BellIcon/BellRingingIcon/XIcon
in PushSubscribe.tsx) but is listed as a devDependency; open package.json and
move "@phosphor-icons/react" from devDependencies into dependencies so it is
installed during production installs (npm install --omit=dev) and available at
build/runtime.
🧹 Nitpick comments (1)
src/components/pages/home/events.tsx (1)
230-242: Nit: unnecessary template literals on static class strings.Lines 231, 236, and 241 use backtick template literals (
`) forclassNamebut contain no interpolations. Regular string quotes would be more conventional.Proposed fix
- <span - className={`whitespace-nowrap rounded bg-green-100 px-2 py-0.5 text-xs text-green-800`} - > + <span className='whitespace-nowrap rounded bg-green-100 px-2 py-0.5 text-xs text-green-800'>Apply the same pattern to the blue and yellow badge spans.
| "@commitlint/config-conventional": "^19.6.0", | ||
| "@eslint/eslintrc": "^3.2.0", | ||
| "@eslint/js": "^9.18.0", | ||
| "@phosphor-icons/react": "^2.1.10", |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# First, let's look at the package.json to see the current dependency structure
cat -n package.json | head -50Repository: FOSSUChennai/Communities
Length of output: 2385
🏁 Script executed:
# Search for all imports and usages of `@phosphor-icons/react` in the codebase
rg "@phosphor-icons/react" --type-add 'pkg:package.json' -t pkg -t ts -t tsx -nRepository: FOSSUChennai/Communities
Length of output: 95
🏁 Script executed:
# Check specifically for events.tsx and MapPin usage
fd -e tsx -e ts | xargs grep -l "MapPin" 2>/dev/nullRepository: FOSSUChennai/Communities
Length of output: 145
🏁 Script executed:
# Look at the events.tsx file to see what's actually imported
find . -name "events.tsx" -o -name "events.ts" | head -5 | xargs cat -n 2>/dev/nullRepository: FOSSUChennai/Communities
Length of output: 14191
🏁 Script executed:
# Check if MapPin is used in archive.tsx and any other files
rg "MapPin" -t ts -t tsx -nRepository: FOSSUChennai/Communities
Length of output: 95
🏁 Script executed:
# Also check for any other `@phosphor-icons` imports
rg "from '@phosphor-icons" -nRepository: FOSSUChennai/Communities
Length of output: 609
@phosphor-icons/react is a runtime dependency and must be in dependencies.
This package provides UI icon components used at runtime across multiple client components: MapPin in events.tsx and archive.tsx, CalendarX in no-events-card.tsx, RssSimple in header.tsx, and BellIcon/BellRingingIcon/XIcon in PushSubscribe.tsx. Moving it to devDependencies breaks production builds in environments that run npm install --omit=dev before next build.
Proposed fix
Move @phosphor-icons/react to dependencies:
"dependencies": {
+ "@phosphor-icons/react": "^2.1.10",
"framer-motion": "^12.34.0",
...
},
"devDependencies": {
...
- "@phosphor-icons/react": "^2.1.10",
...
},🤖 Prompt for AI Agents
In `@package.json` at line 34, The package "@phosphor-icons/react" is required at
runtime by UI components (e.g., MapPin in events.tsx and archive.tsx, CalendarX
in no-events-card.tsx, RssSimple in header.tsx, and
BellIcon/BellRingingIcon/XIcon in PushSubscribe.tsx) but is listed as a
devDependency; open package.json and move "@phosphor-icons/react" from
devDependencies into dependencies so it is installed during production installs
(npm install --omit=dev) and available at build/runtime.
1 participant
Adding techKoodaram community based in South Tamil Nadu with details and website link, also added the upcoming Build Day event planned on 28th February.
Changes:
Summary by CodeRabbit
Bug Fixes
New Features
Chores