Skip to content

Feature/refonte architecture - #6

Open
Denismaka wants to merge 12 commits into
congodevelopersclub:mainfrom
Denismaka:feature/refonte-architecture
Open

Feature/refonte architecture#6
Denismaka wants to merge 12 commits into
congodevelopersclub:mainfrom
Denismaka:feature/refonte-architecture

Conversation

@Denismaka

Copy link
Copy Markdown

Contexte

Le point de départ était simple : proposer un header et un footer plus modernes pour congodevelopers.club/fr, que je trouvais daté.

En creusant le code actuel pour m'en inspirer, j'ai constaté qu'il ne respecte pas la Stack Technique & Architecture qu'on a validée ensemble (document de référence). Cette PR corrige ça.

Constat sur le code actuel (main)

Point de l'architecture Attendu Constaté
Monorepo (apps/ + packages/) Oui Absent, une seule application à plat
Feature-Sliced Design Oui Absent, structure par type de fichier
Zustand (état UI local) Oui Redux Toolkit utilisé à la place
TanStack Query (données API) Oui Absent, aucune dépendance
Storybook Oui Absent
Docker (dev) Oui Absent
Palette (#3C7EA9 / #50AA79 / #F5A623) Oui Autres couleurs utilisées (#3b82f6, #10B981, #8b5cf6)
Server Actions (BFF) Oui Un seul cas, mal implémenté (appel GitHub sans token, 60 req/h)
Next.js / TypeScript Oui Respecté
Shadcn/UI Oui Outillage en place, quasiment pas utilisé

Ce que propose cette PR

Une base qui respecte réellement la stack validée, en reprenant le contenu et les intentions de design du site actuel, avec des améliorations :

  • Monorepo npm workspaces : apps/web + packages/{ui,types,api-client,config-eslint,config-typescript}
  • Feature-Sliced Design dans apps/web/src (app/, views/, widgets/, features/, entities/, shared/)
  • Shadcn/UI réellement utilisé (Button, Card, Carousel, TiltCard) + Storybook avec les stories Default/Loading/Disabled/With Icon
  • TanStack Query pour toutes les données serveur (ex. contributeurs GitHub, événements, articles), Zustand uniquement pour l'état d'interface (thème, langue, menu mobile)
  • Server Actions comme seul point de passage vers un backend (@cdc/api-client, server-only)
  • Contributions GitHub réelles, agrégées depuis les dépôts publics de congodevelopersclub (pas de chiffres inventés)
  • Docker pour le développement (docker-compose.yml, Dockerfile.dev) — pas obligatoire en prod, comme précisé dans le document d'architecture
  • Palette de marque correcte (#FFFFFF / #3C7EA9 / #50AA79 / #F5A623)
  • Pages Accueil, À propos, Activités, Événements, Blog, en français et anglais

Détails complets dans docs/Architecture.md.

Ce qui reste ouvert

Cette PR ne prétend pas remplacer le travail déjà fait sans discussion. Deux options possibles, à trancher en équipe :

  1. On repart de cette base, déjà conforme à l'architecture validée, et on continue dessus.
  2. On garde le code actuel et on le met progressivement en conformité (Redux vers Zustand, structure plate vers FSD, etc.).

Je suis dispo pour en discuter avant tout merge.

@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

@Denismaka is attempting to deploy a commit to the Patrick Nahayo's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify

netlify Bot commented Jul 15, 2026

Copy link
Copy Markdown

Deploy Preview for congodevelopersclub failed. Why did it fail? →

Name Link
🔨 Latest commit 606e0db
🔍 Latest deploy log https://app.netlify.com/projects/congodevelopersclub/deploys/6a58d288eef21f000805baa3

@@ -0,0 +1,7 @@
export const navLinks = [

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool est ce du next.js

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oui est ce du nest.js

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oui

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.

2 participants