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

Proposer une adresse : le bouton devrait etre visible sur la carte du formulaire #1061

Merged
merged 11 commits into from
Nov 26, 2024

Conversation

fabienheureux
Copy link
Member

@fabienheureux fabienheureux commented Nov 25, 2024

Description succincte du problème résolu

Le footer de la version formulaire devrait fonctionner différemment

  • Visible uniquement lorsque la carte est affichée
  • Le bouton proposer une adresse est masqué si le formulaire n'est pas ouvert
    Note : cf Supprimer les absolute de la version formulaire  #1020 il serait idéal de gérer ça directement dans le templating plutôt qu'en CSS.
    Une amélioration pour plus tard 😇

Aussi

  • ⚠️ J'ai identifié un soucis sur la manière dont étaient découpés les css entre assistant et carte. j'en ai profité pour corriger cela.
  • J'ai identifié un soucis avec la légende qui était visible sur la home de la carte. Je corrige ça proprement sans utiliser des z-index élevés qui ne sont jamais très maintenances
    image
  • Ça m'a fait comprendre qu'on avait pas mal de code inutile dans le javascript et des z-index également, je les ai supprimés et ai réorganisé les templates à la place.
    En effet, le template charge dans un turbo frame, donc pas besoin de mettre à jour les classes depuis le javascript

Tooling/tests

  • J'ai couvert les bugs par des tests e2e
  • J'ai ajouté le commit du renommage qfdmo > qf dans le fichier git blame ignore rev pour simplifier le debuguage et éviter que ce commit ne pop dans nos git blame

Type de changement :

  • Bug fix
  • Nouvelle fonctionnalité
  • Mise à jour de données / DAG
  • Les changements nécessitent une mise à jour de documentation
  • Refactoring de code (explication à retrouver dans la description)

Auto-review

Les trucs à faire avant de demander une review :

  • J'ai bien relu mon code
  • La CI passe bien
  • En cas d'ajout de variable d'environnement, j'ai bien mis à jour le .env.template
  • J'ai ajouté des tests qui couvrent le nouveau code

Comment tester

  • Aller sur /formulaire
  • Le bouton proposer une adresse ne doit pas être présent
  • Faire une recherche
  • Il doit etre présent

@fabienheureux fabienheureux marked this pull request as ready for review November 25, 2024 10:55
@fabienheureux fabienheureux requested a review from a team as a code owner November 25, 2024 10:55
@fabienheureux fabienheureux requested review from kolok and maxcorbeau and removed request for a team November 25, 2024 10:55
@fabienheureux fabienheureux changed the title Proposer une adresse should be visible on carte formulaire Proposer une adresse : le bouton devrait etre visible sur la carte du formulaire Nov 25, 2024
@fabienheureux fabienheureux assigned fabienheureux and unassigned kolok Nov 25, 2024
@fabienheureux fabienheureux force-pushed the fiche-detaillee-feedbacks-3 branch from 507f634 to c6b0a49 Compare November 25, 2024 12:47
Copy link
Contributor

@kolok kolok left a comment

Choose a reason for hiding this comment

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

cool !

@fabienheureux fabienheureux enabled auto-merge (squash) November 25, 2024 15:39
@fabienheureux fabienheureux merged commit d1c12e2 into main Nov 26, 2024
7 checks passed
@fabienheureux fabienheureux deleted the fiche-detaillee-feedbacks-3 branch November 26, 2024 08:09
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