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

Correction et refacto de l'affichage des questions #3427

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

Conversation

liliced
Copy link
Contributor

@liliced liliced commented Feb 13, 2025

#3417

  • Refactorisation du composant Banner

    • déplacement dans le dossier Simulation
    • renommage cohérent
    • déplacement de la gestion de la disparition après la 1ère étape dans les parents
  • Refactorisation de SelectSimulationYear

    • déplacement dans le dossier Simulation
    • renommage cohérent
  • Refactorisation de SimulationPréremplieBanner

    • déplacement dans le dossier Simulation
    • ne s'affiche que lorsqu'il n'y a pas la sélection d'entreprise (peu pertinent sinon)
    • reste affiché même après le début de la simulation (car toujours pertinent)
  • Refactorisation de PreviousSimulationBanner

    • déplacement dans le dossier Simulation
    • déplacement de la gestion de la disparition après la 1ère étape dans le parent Simulation
  • Ajout d'une variable exceptions à firstStepCompletedSelector, transmise par Simulation et remplie au besoin par le simulateur (cessation d'activité, dividendes, indépendant, lodeom et RGCP)

  • Correction de 2 avertissements console (prop inconnue et absence de key)

  • Ajout de tests sur les simulateurs génériques

  • Ajout de paramètres aux tests des simulateurs génériques

Copy link

netlify bot commented Feb 13, 2025

Deploy Preview for nostalgic-mahavira-52b682 canceled.

Name Link
🔨 Latest commit 91b7ed8
🔍 Latest deploy log https://app.netlify.com/sites/nostalgic-mahavira-52b682/deploys/67af0a54b0334300087615eb

@liliced liliced linked an issue Feb 13, 2025 that may be closed by this pull request
@github-actions github-actions bot temporarily deployed to 3427 February 13, 2025 21:33 Inactive
@github-actions github-actions bot temporarily deployed to 3427 February 13, 2025 21:33 Inactive
Copy link

github-actions bot commented Feb 13, 2025

@liliced liliced force-pushed the fix/3417-questions-artiste-auteur branch from 80f2e3c to 145d198 Compare February 14, 2025 08:38
@github-actions github-actions bot temporarily deployed to 3427 February 14, 2025 08:43 Inactive
@github-actions github-actions bot temporarily deployed to 3427 February 14, 2025 08:43 Inactive
@liliced liliced force-pushed the fix/3417-questions-artiste-auteur branch from 145d198 to 91b7ed8 Compare February 14, 2025 09:18
@liliced liliced requested a review from glopezrios February 14, 2025 09:19
@github-actions github-actions bot temporarily deployed to 3427 February 14, 2025 09:22 Inactive
export const runSimulateurTest = (simulateur: Simulateur) => {
export const runSimulateurTest = (
simulateur: Simulateur,
avecCharges = false,
Copy link
Contributor

Choose a reason for hiding this comment

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

Comme on risque de multiplier les options, préférer peut-être un objet options :

export const runSimulateurTest = (
	simulateur: Simulateur,
	{ avecCharges = false }: RunSimulateurOptions,
	beforeAction = () => {}
) => {

ce qui facilite aussi la lecture à l’utilisation en étant explicite :

runSimulateurTest('profession-liberale/medecin', { avecCharges: true})

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pourquoi ne pas mettre beforeAction dans les options aussi ?

Copy link
Contributor

@glopezrios glopezrios left a comment

Choose a reason for hiding this comment

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

C'est tout bon, les questions sont bien revenues sur le simulateur pour artiste-auteur!

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.

Sur le simulateur pour artiste-auteur, les questions ont disparu
3 participants