bug fixes

[SimonBurmer]

Summary by CodeRabbit

• New Features

  • Added feature flag to control Advisory Board section visibility
  • Added Unsplash image source support

• Refactor

  • Migrated image elements to Next.js Image component for optimization
  • Simplified numeric animation logic
  • Updated gallery rendering system

• Style

  • Removed hover effects from mission partner cards
  • Adjusted call-to-action container padding

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
start-munich	Error		Apr 4, 2026 10:01am

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 62699a9a-c385-465b-87c8-bbb848cac08c

📥 Commits

Reviewing files that changed from the base of the PR and between c0e368a and 3a52ee8.

📒 Files selected for processing (5)

• app/about-us/page.tsx
• app/for-partners/page.tsx
• components/TestimonialsSection.tsx
• lib/useAnimatedNumber.ts
• next.config.js

---

📝 Walkthrough

Walkthrough

This pull request modernizes image handling across multiple pages by adopting Next.js Image components, introduces a feature flag for conditional rendering of the Advisory Board section, refactors animation logic to remove gate-keeping mechanisms, updates the Mission Partners data model with structured description parts, and expands remote image source allowlisting.

Changes

Cohort / File(s)	Summary
Image Component & Configuration app/for-partners/page.tsx, components/TestimonialsSection.tsx, next.config.js	Migrated <img> elements to Next.js Image component with fill layout and object-cover styling; updated hero image wrapper to position: relative; marked TUM logo as unoptimized; added images.unsplash.com to remotePatterns configuration.
About Us Page Enhancements app/about-us/page.tsx	Restructured CDTM Mission Partner description from single string to descriptionParts array for selective text highlighting; introduced showAdvisoryBoard feature flag gating entire Advisory Board section; removed hover/transition styling from mission partner cards and images; updated partner name text formatting.
For Partners Page & Testimonials app/for-partners/page.tsx	Disabled TestimonialsSection with hardcoded conditional; refactored event gallery rendering from index-based slicing to visibility-class toggling on all photos; adjusted CTA container padding from p-10 md:p-16 to p-8 md:p-10.
Animation Logic Refactoring lib/useAnimatedNumber.ts	Removed useRef-based gating that prevented re-animation; simplified useEffect to immediately exit on loading or target <= 0; restructured animation setup to run conditionally on each effect invocation; updated interval cleanup to always return clearance function when timer is created.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Fix: about us page #42: Overlaps on Mission Partners image handling and card rendering logic in app/about-us/page.tsx.
• feat: restyle for-partners page with modern card design #30: Modifies app/for-partners/page.tsx testimonial handling and gallery rendering alongside TestimonialsSection styling changes.
• feat: make Member Journey ROUND #34: Changes directly affect lib/useAnimatedNumber.ts hook implementation that was recently integrated elsewhere in the codebase.

Poem

🐰 Images optimized with Next.js care,
Feature flags floating through the air,
Gallery logic dances anew,
Animation gates bid their farewell too,
Mission Partners shine in parts so bright!

✨ Finishing Touches

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch fixes

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}