-
Notifications
You must be signed in to change notification settings - Fork 87
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
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for nostalgic-mahavira-52b682 canceled.
|
🚀 La branche est déployée !
|
80f2e3c
to
145d198
Compare
145d198
to
91b7ed8
Compare
export const runSimulateurTest = (simulateur: Simulateur) => { | ||
export const runSimulateurTest = ( | ||
simulateur: Simulateur, | ||
avecCharges = false, |
There was a problem hiding this comment.
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})
There was a problem hiding this comment.
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 ?
There was a problem hiding this 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!
#3417
Refactorisation du composant
Banner
Simulation
Refactorisation de
SelectSimulationYear
Simulation
Refactorisation de
SimulationPréremplieBanner
Simulation
Refactorisation de
PreviousSimulationBanner
Simulation
Simulation
Ajout d'une variable
exceptions
àfirstStepCompletedSelector
, transmise parSimulation
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 dekey
)Ajout de tests sur les simulateurs génériques
Ajout de paramètres aux tests des simulateurs génériques