Skip to content

Conversation

kara22
Copy link
Contributor

@kara22 kara22 commented Sep 11, 2025

#11922

Interface administrateur

Titre d'identité :

image

RIB :

image

Autres champs :

image

Interface usager :

image

@kara22 kara22 marked this pull request as draft September 11, 2025 21:15
Copy link

codecov bot commented Sep 11, 2025

Codecov Report

❌ Patch coverage is 94.36620% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.59%. Comparing base (52eeaa3) to head (43624be).

Files with missing lines Patch % Lines
app/components/attachment/edit_component.rb 93.54% 4 Missing ⚠️
app/models/procedure_revision.rb 42.85% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12064      +/-   ##
==========================================
+ Coverage   89.57%   89.59%   +0.02%     
==========================================
  Files        1334     1334              
  Lines       30257    30391     +134     
  Branches     6341     6413      +72     
==========================================
+ Hits        27103    27230     +127     
- Misses       3154     3161       +7     

☔ 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.

@kara22
Copy link
Contributor Author

kara22 commented Sep 11, 2025

@tchak avant que je fasse le dernier commit de tests, j'aimerais bien avoir ton retour a savoir s’il y a des choses à optimiser / changer, merci d'avance.

@kara22 kara22 force-pushed the feat/11922 branch 3 times, most recently from 5d87e2a to 02e20a7 Compare September 18, 2025 13:42
end

def filter_type_champ(type_champ)
return false if type_champ == TypeDeChamp.type_champs.fetch(:titre_identite)
Copy link
Member

Choose a reason for hiding this comment

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

il va se passer quoi pour les démarches qui ont encore un titre_identite ? Je pense qu'il faut faire comme pour le type number

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui , comme pour number, on masque seulement l’option dans la liste des types à créer.
Les démarches qui ont déjà un titre identité continuent de fonctionner et s’éditent normalement. On peut changer vers un autre type, mais on ne peut plus en créer ni re modifier vers l'ancien titre identité.

C'est bon pour toi @tchak ?

Copy link
Member

Choose a reason for hiding this comment

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

Le comportement que tu décris est bon, mais je pense que ton code ne fait pas ce que tu dis. Si tu essaies de modifier une démarche avec un type de champ titre_identite, tu vas avoir un select vide. Il faut exclure titre_identite de la liste que si le type de champ n'est pas déjà de ce type

@kara22 kara22 force-pushed the feat/11922 branch 2 times, most recently from 1784836 to 26508ba Compare September 19, 2025 07:49
@kara22 kara22 marked this pull request as ready for review September 19, 2025 08:24
@kara22 kara22 requested a review from tchak September 19, 2025 08:24
@kara22
Copy link
Contributor Author

kara22 commented Sep 19, 2025

@marleneklok pour info

@marleneklok
Copy link

@marleneklok pour info

Hello @kara22
Voici quelques retours UX/UI :

image

image

image

image

image

Je suis dispo pour qu'on s'en parle / que tu me fasses une petite démo si tu veux aujourd'hui ;)

@kara22 kara22 force-pushed the feat/11922 branch 7 times, most recently from 5105ad0 to 594bbd2 Compare October 6, 2025 08:54
@kara22
Copy link
Contributor Author

kara22 commented Oct 6, 2025

Titre identité :

image

RIB :

image

Non spécifié :

image

Interface usager :

image

@kara22 kara22 force-pushed the feat/11922 branch 3 times, most recently from cb20989 to cf78bfe Compare October 13, 2025 11:31
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