Skip to content

feat(storybook): update storybook from 6 to 10 - #477

Open
lsagetlethias wants to merge 1 commit into
mainfrom
fix/post-revert
Open

feat(storybook): update storybook from 6 to 10#477
lsagetlethias wants to merge 1 commit into
mainfrom
fix/post-revert

Conversation

@lsagetlethias

Copy link
Copy Markdown
Collaborator

@garronej Here a fix for the sidemenu.

I had to copy small parts of the original code because not everything is exposed on their side.
Should be better now for responsive and with firefox.

Commit are took before the last bump and squashed together.

Copilot AI review requested due to automatic review settings January 29, 2026 13:19
@lsagetlethias
lsagetlethias requested review from garronej and removed request for Copilot January 29, 2026 13:20
@lsagetlethias

Copy link
Copy Markdown
Collaborator Author

Up @garronej, do you need any help on this?

Comment thread package.json
"next": "13.5.1",
"parse-numeric-range": "^1.3.0",
"patch-package": "^8.0.0",
"polished": "^4.3.1",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

necessary ?

Comment thread package.json
"tss-react": "^4.9.1",
"type-route": "^1.0.1",
"typescript": "^4.9.1",
"vite": "^7.0.0",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

necessary ?

@kevbarns

Copy link
Copy Markdown
Collaborator

@lsagetlethias @garronej @revolunet Cette PR est ouverte depuis janvier 2026 et bloque toute évolution de la Storybook (83 fichiers, +2778/-9514). Elle est verte en CI et sans conflit aujourd'hui, mais chaque semaine qui passe augmente le risque de rebase.

Deux questions de @revolunet sont restées sans réponse depuis le 19/08. Éléments de réponse pour les débloquer :

  • package.json:119polished : utilisé une seule fois dans tout le diff, import { transparentize } from "polished" dans .storybook/DocsContainer/DocsPageWrapper.tsx:6, pour deux appels lignes 85-86. La dépendance est donc réelle, mais remplaçable par un helper local de trois lignes si vous préférez ne pas l'ajouter.
  • package.json:131vite : c'est une peer dependency obligatoire de @storybook/react-vite@10.2.1 (^5.0.0 || ^6.0.0 || ^7.0.0). Elle est nécessaire.

@lsagetlethias pouvez-vous confirmer ces deux points, ou trancher sur polished ?

@garronej @revolunet reste la question de fond : est-ce que la migration Storybook 6 → 10 est validée dans son principe ? Si oui, il faudrait la merger vite compte tenu de sa taille. Si non ou si elle doit attendre, autant le dire pour que @lsagetlethias arrête de la maintenir à jour.

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.

3 participants