incubateur-ademe / bilansclimatsimplifies

MIT License
1 stars 1 forks source link

Formulaire inscription entreprise (Fanny) #27

Closed Clem-mrtn closed 2 years ago

Clem-mrtn commented 2 years ago

Question : Quel contrôle est en place aujourd’hui sur l’année complétée ? @florianpanchout La règle côté Bilans GES, c’est que l’année de reporting doit être l’année n, n-1 ou n-2 par rapport à l’année de publication. On peut la reprendre ici et indiquer ça en sous-titre « année n, n-1 ou n-2 par rapport à l’année en cours » ?

Question : : Pour la Région, quel est le numéro attendu ? Le numéro de la région ou le code INSEE de la région ? S’il s’agit du code INSEE, il faut a minima le préciser dans le sous-titre. Est-il compliqué de mettre plutôt une liste déroulante avec les noms des différentes régions plutôt que de demander un numéro ?

hfroot commented 2 years ago

@florianpanchout est-ce que tu veux que j'ajoute des vérifications pour l'année et nombre salariés côté back aussi ?

florianpanchout commented 2 years ago

Yes, ça me semble pertinent d'avoir toutes les vérifications en back (en plus du front)

hfroot commented 2 years ago

Sur la page d’accueil, c'est écrit que l'outil est pour les entreprises entre 50 et 500 salariés - alors est-ce qu'on veut ajouter une vérification pour >=50 aussi que <=500 ?

Clem-mrtn commented 2 years ago

Carrément, oui je pense que c'est bien d'avoir les deux vérifications !

Clem-mrtn commented 2 years ago

J'ai un bug , j'ai pu créer un premier Bilan avec mon numéro de Siren. Ensuite j'ai supprimé ce bilan pour pouvoir en réediter un nouveau avec le même numéro de siren et tester un nouveau parcours utilisateur et je suis coincée le formulaire me bloque avec un message d'erreur : image

=> Je ne sais pas si c'est parce qu'il n'a pas encore enregistré qu'il n'y avait plus de bilan sous ce numéro ou alors il y a un autre bug. PI j'ai tenté de laisser mon premier bilan existant, et d'en créer un nouveau en changeant uniquement le chiffre de fin pour le SIREN mais pareil j'était bloquée... (je me suis dis que ce Siren ne devait donc pas exister)

hfroot commented 2 years ago

@Clem-mrtn je pense c'est au cause de l'année que tu utilises - il y a maintenant de la validation pour voir si l'année est n-2, n-1, ou n - 2018 c'est n-3.

florianpanchout commented 2 years ago

@hfroot Si j'essaie de créer un bilan avec un couple siren / année déja utilisé, l'api me renvoie une erreur 400 sans plus de détail. Possible de me préciser l'erreur ?

hfroot commented 2 years ago

@florianpanchout maintenant on retourne 'nonFieldErrors': ['Les champs siren, annee doivent former un ensemble unique.'] si il y a une duplication

florianpanchout commented 2 years ago

Possible de mettre un message plus grand public ? Du style Un bilan avec ce couple SIREN / Année de reporting existe déjà. (@martinregner @Clem-mrtn ou autre chose si vous avez plus compréhensible)

Comme ça je peux afficher directement la réponse de l'API sans passer par un dictionnaire d'équivalence en front

hfroot commented 2 years ago

J'ai mis à jour le message, ça sera prêt dans quelques minutes

florianpanchout commented 2 years ago

C'est en ligne