English Turkish translation added.

[HyperZno]

I couldn't do a test, sorry. This is my first time using GitHub and I still haven't gotten used to it :(

Summary by CodeRabbit

• New Features

  • Philosophy page now supports English and Turkish languages with an integrated language toggle.
  • Language preferences are automatically saved for future visits.

• Improvements

  • All content on the Philosophy page, including headings, text, and calls-to-action, is now dynamically translated based on selected language.
  • Updated navigation to include new join path.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The Philosophy page now includes comprehensive internationalization support with English and Turkish translations. The change introduces language state management with localStorage persistence, a language toggle UI, and replaces all hardcoded content strings with dynamic translation lookups throughout the page structure.

Changes

Cohort / File(s)

Change Summary

Internationalization implementation pages/philosophy.js

Added React hooks (useEffect, useMemo, useState) and Flex component import. Introduced language state management with localStorage persistence. Created comprehensive copy object containing English and Turkish translations for all content. Replaced hardcoded Meta title/description, hero section, header/Seal content, all section headings and bodies (sections 1–6), and CTA block with dynamic translated values. Added language toggle UI (TR/EN buttons) near hero. Updated component rendering logic to pull strings from translation object; preserved HTML element combinations (strong, Link) in translated content for sections 3 and 5.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Poem

🐰 The Philosophy page speaks in tongues,
With English words and Turkish songs,
Toggle buttons dance in light,
localStorage keeps translations bright—
A rabbit's internationalization delight! ✨

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: adding English-Turkish translation support to the Philosophy page with full i18n implementation.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[coderabbitai]

Actionable comments posted: 4

🧹 Nitpick comments (2)

pages/philosophy.js (2)

123-201: Line length exceeds 80 characters.

Multiple lines in the translation object exceed the 80-character limit specified in the coding guidelines. Consider breaking long strings across multiple lines or extracting translations to a separate file.

🔎 Example refactor for readability

-        metaDescription:
-          'Read about Happy Hacking Space, a network of hackers. We want to make building apps and websites accessible to everyone.',
+        metaDescription: 'Read about Happy Hacking Space, a network of ' +
+          'hackers. We want to make building apps and websites ' +
+          'accessible to everyone.',

Alternatively, consider moving translations to a separate locales/ directory for better maintainability as the translation set grows.

---

353-353: Variable shadowing: t shadows translation function.

The Theme UI callback parameter t shadows the translation helper t(key) defined on line 203. While this works because of scope, it reduces readability.

🔎 Proposed rename for clarity

-backgroundImage: (t) => t.util.gx('orange', 'red'),
+backgroundImage: (theme) => theme.util.gx('orange', 'red'),

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d94fa83 and 85360ca.

📒 Files selected for processing (1)

• pages/philosophy.js

🧰 Additional context used

📓 Path-based instructions (3)

**/*.{js,jsx,ts,tsx}

📄 CodeRabbit inference engine (AGENT.md)

**/*.{js,jsx,ts,tsx}: Use relative imports (e.g., '../components/nav') for importing modules
Use Theme UI components (e.g., { Box, Text }) for UI elements
Use single quotes for strings
Do not use semicolons at the end of statements
Do not use trailing commas in arrays, objects, or function parameters
Use camelCase for variable and function names
Use PascalCase for component names
Use Theme UI for consistent styling with the Hack Club theme

Files:

• pages/philosophy.js

**/*.{js,jsx,ts,tsx,md,mdx}

📄 CodeRabbit inference engine (AGENT.md)

Limit lines to 80 characters in width

Files:

• pages/philosophy.js

pages/**/*.{js,jsx,ts,tsx,mdx}

📄 CodeRabbit inference engine (AGENT.md)

Use the pages/ directory structure for Next.js routing

Files:

• pages/philosophy.js

🔇 Additional comments (3)

pages/philosophy.js (3)

3-7: LGTM!

Imports are correctly structured using Theme UI components and React hooks as needed for the i18n implementation.

---

203-203: LGTM!

The translation helper has a solid fallback chain: current language → English fallback → key itself. This ensures graceful degradation for missing translations.

---

369-377: LGTM!

The CTA button correctly uses target="_blank" with rel="noopener noreferrer" for security, and the translated button text is properly applied.