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

feat(simulateurs): Ajoute la sélection de son entreprise en première étape des simulateurs [alt alt version] #3201

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

liliced
Copy link
Contributor

@liliced liliced commented Nov 5, 2024

Résoud #1633

@liliced liliced requested a review from JalilArfaoui November 5, 2024 11:33
@liliced liliced self-assigned this Nov 5, 2024
Copy link

netlify bot commented Nov 5, 2024

Deploy Preview for nostalgic-mahavira-52b682 canceled.

Name Link
🔨 Latest commit 46370d7
🔍 Latest deploy log https://app.netlify.com/sites/nostalgic-mahavira-52b682/deploys/67517d0a9734380008d192ab

Copy link

github-actions bot commented Nov 5, 2024

♿ Checklist accessibilité :

Quelques points à vérifier pour maintenir une bonne accessibilité du site :

  • les labels des boutons et des liens sont descriptifs
  • un aria-label est ajouté pour préciser l'action d'un bouton ou d'un lien pour les lecteurs d'écran
  • les couleurs du texte est suffisamment constratée par rapport à la couleur de background
  • la fonctionnalité ajoutée est utilisable en navigant au clavier et avec un lecteur d'écran
  • les changements de contexte sont évités ou au minimum annoncés

@liliced liliced linked an issue Nov 5, 2024 that may be closed by this pull request
@github-actions github-actions bot temporarily deployed to 3201 November 5, 2024 11:38 Inactive
@github-actions github-actions bot temporarily deployed to 3201 November 5, 2024 11:38 Inactive
Copy link

github-actions bot commented Nov 5, 2024

@liliced liliced force-pushed the feat/1633-ajout-recherche-entreprise-simulateurs-ter branch from 2c3e3e5 to 41c5945 Compare November 5, 2024 11:39
@github-actions github-actions bot temporarily deployed to 3201 November 5, 2024 11:47 Inactive
@github-actions github-actions bot temporarily deployed to 3201 November 5, 2024 11:47 Inactive
@glopezrios
Copy link
Collaborator

Suite à un échange avec @VeroniqueR75, constat d'une limite de cette proposition: on peut faire une simulation pour un type d'entreprise (par exemple SAS(U)) alors qu'on a renseigné son entreprise qui est d'un autre type (par exemple, auto-entreprise).

Besoin d'afficher un message d'alerte quand c'est le cas ? Et/ou rediriger vers le bon simulateur ?

@liliced liliced force-pushed the feat/1633-ajout-recherche-entreprise-simulateurs-ter branch from cf984f2 to 439aea0 Compare December 3, 2024 15:22
@liliced liliced marked this pull request as ready for review December 3, 2024 15:24
@liliced liliced force-pushed the feat/1633-ajout-recherche-entreprise-simulateurs-ter branch from 439aea0 to e55b840 Compare December 3, 2024 16:22
@github-actions github-actions bot temporarily deployed to 3201 December 3, 2024 16:27 Inactive
@github-actions github-actions bot temporarily deployed to 3201 December 3, 2024 16:27 Inactive
…reprise sélectionnée ne correspond pas au simulateur
@liliced liliced force-pushed the feat/1633-ajout-recherche-entreprise-simulateurs-ter branch from e55b840 to 46370d7 Compare December 5, 2024 10:14
@github-actions github-actions bot temporarily deployed to 3201 December 5, 2024 10:19 Inactive
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.

Ajouter un composant recherche entreprise sur tous les simulateurs
3 participants