Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Met à jour les cartes sollicitation #3745

Merged
merged 6 commits into from
Jan 14, 2025
Merged

Conversation

LucienMLD
Copy link
Collaborator

closes #3730

@LucienMLD LucienMLD force-pushed the lucien/carte-sollicitation branch from 2bbaed0 to ecdfcd6 Compare December 18, 2024 12:26
@LucienMLD LucienMLD requested a review from clairezed December 18, 2024 13:20
@LucienMLD LucienMLD force-pushed the lucien/carte-sollicitation branch from ecdfcd6 to d42e48d Compare December 18, 2024 13:23
Copy link
Collaborator

@clairezed clairezed left a comment

Choose a reason for hiding this comment

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

Y'a des trucs cassés, suite à l'utilisation de stream, vraisemblablement :

  • Chez moi, le clic sur le lien titre ne fonctionne pas : disparition de la carte remplacé par "Content missing"
  • idem au clic sur "historique de l'entreprise, sur le bouton "analyse en cours"...
  • l'ajout de commentaire plante aussi
  • pas de changement de page automatique si on dépile tous les besoins. Normal avec la façon dont on utilise les stream, on a le même soucis à d'autres endroits. Mais ce serait bien qu'on trouve un moyen de mieux le gérer

@LucienMLD LucienMLD force-pushed the lucien/carte-sollicitation branch from d42e48d to f17027c Compare December 20, 2024 07:31
@LucienMLD
Copy link
Collaborator Author

C'est bon pour les liens et les commentaires.
Pour le rechargement de page je fais un ticket, il faudra revoir un peu comment sont faites les pages ou il y a du turbo et de la pagination.

Copy link
Collaborator

@clairezed clairezed left a comment

Choose a reason for hiding this comment

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

ok, ça semble marcher !
Mais devoir disabler en masse turbo m'interroge. J'aimerais bien trouver/prendre le temps de me former un peu mieux à turbo et tout son écosystème, histoire de comprendre comment l'utiliser au mieux dans notre contexte

@LucienMLD
Copy link
Collaborator Author

D'après la compréhension que j'ai de turbo il faudrait ajouter des frame un peu partout pour faire une application style One Page. Pour le moment c'est pas trop la priorité et j'essaie d'ajouter du Turbo petit à petit. Ca me parait beaucoup de boulot pour tout faire d'un coup

@LucienMLD LucienMLD force-pushed the lucien/carte-sollicitation branch 4 times, most recently from b7505e8 to 0e4f607 Compare January 6, 2025 13:46
Copy link
Collaborator

@clairezed clairezed left a comment

Choose a reason for hiding this comment

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

Il semble que ça marche, mais je me questionne quand même : ça fait un peu le truc à moitié fait avec les compteurs qui se décrémentent pas. A voir avec les collègues si ça leur convient à l'usage

A creuser + tard avec ce type de ressource ?

@LucienMLD LucienMLD force-pushed the lucien/carte-sollicitation branch from 0e4f607 to 7bb68e9 Compare January 14, 2025 09:09
@LucienMLD LucienMLD merged commit 8e48d3a into main Jan 14, 2025
9 checks passed
@LucienMLD LucienMLD deleted the lucien/carte-sollicitation branch January 14, 2025 09:33
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.

Nouvelle carte de sollicitation pour se simplifier la vie !
2 participants