Skip to content

ETQ usager on ne me demande plus ma civilité par défaut#11489

Merged
tchak merged 1 commit into
demarche-numerique:mainfrom
tchak:fix-civilite-select
Jul 3, 2025
Merged

ETQ usager on ne me demande plus ma civilité par défaut#11489
tchak merged 1 commit into
demarche-numerique:mainfrom
tchak:fix-civilite-select

Conversation

@tchak

@tchak tchak commented Mar 18, 2025

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Mar 25, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.91%. Comparing base (4d129c6) to head (5bf1c2b).
Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11489      +/-   ##
==========================================
- Coverage   89.91%   89.91%   -0.01%     
==========================================
  Files        1291     1291              
  Lines       28370    28369       -1     
  Branches     5873     5873              
==========================================
- Hits        25510    25509       -1     
  Misses       2860     2860              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tchak
tchak force-pushed the fix-civilite-select branch 3 times, most recently from 1e40c91 to 2a0d096 Compare March 25, 2025 13:51
@colinux colinux changed the title feat(identite): remove gender from identite bloc ETQ usager on ne me demande plus ma civilité par défaut Mar 27, 2025

@colinux colinux left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ça me paraît nickel à première vue, mais je continue de creuser dans ma tête aux éventuelles autres implications.

Je pense par exemple aux filtres instructeurs et exports, mais surtout aux démarches actuelles qui en auraient vraiment besoin : le temps que les admins réagissent on aurait des dossiers déposé sans, est-ce qu'on est ok avec le fait que ça impliquera aux instructeurs de demander une correction et que ça risque de noyer le support ?
Est-ce qu'on ne ferait pas une comm spécifique avant de déployer ?

@colinux

colinux commented May 26, 2025

Copy link
Copy Markdown
Member

Pour suivi depuis qu'on a mergé l'autre PR avec l'attribut translate="no", sur les 2 derniers mois il n'y a pas un seul Individual#gender qui déraille.

@tchak
tchak force-pushed the fix-civilite-select branch 4 times, most recently from 737c983 to 3e5f389 Compare June 2, 2025 10:51
@tchak

tchak commented Jun 2, 2025

Copy link
Copy Markdown
Member Author

@colinux j'ai mis à jour la PR avec un mécanisme qui fait que les nouvelles démarches n'auront plus le champ gender, mais les anciennes ne sont pas affectées. Il reste qu'il faudra préparer une com

@tchak
tchak force-pushed the fix-civilite-select branch from 3e5f389 to d1cf7f3 Compare June 2, 2025 15:30
@tchak
tchak force-pushed the fix-civilite-select branch from d1cf7f3 to f2a75ec Compare June 11, 2025 09:36
@tchak
tchak force-pushed the fix-civilite-select branch from f2a75ec to e18b0ff Compare June 23, 2025 13:54

@colinux colinux left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nickel. Est-ce que dans la foulée on ferait pas une seconde migration pour que no_gender passe à true par défaut dans le schéma pour ne pas avoir à hardcoder un no_gender: true aux endroits où on créé la démarche ?

@tchak

tchak commented Jun 25, 2025

Copy link
Copy Markdown
Member Author

@colinux oui, c'est l'idée

@tchak
tchak force-pushed the fix-civilite-select branch from e18b0ff to 6a923db Compare June 25, 2025 10:18
@tchak
tchak force-pushed the fix-civilite-select branch from 6a923db to 5bf1c2b Compare July 2, 2025 15:04
@tchak
tchak added this pull request to the merge queue Jul 3, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 3, 2025
@tchak
tchak added this pull request to the merge queue Jul 3, 2025
Merged via the queue into demarche-numerique:main with commit 28d3395 Jul 3, 2025
29 of 30 checks passed
@tchak
tchak deleted the fix-civilite-select branch July 3, 2025 13:21
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