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(Restitution): allow ReactNode in value #946

Conversation

MartinWeb
Copy link
Contributor

Related issue

Reference to the issue

#945

Description of the issue

Your description

Person(s) for reviewing proposed changes

You can add @mention here

Important

Before creating a pull request run unit tests

$ npm test

# watch for changes
$ npm test -- --watch

# For a specific file (e.g., in packages/context/__tests__/command.test.js)
$ npm test -- --watch packages/action

@MartinWeb MartinWeb force-pushed the feature/reactNodeForRestitutionValue branch from f62f949 to 77cbb3d Compare January 28, 2022 16:06
@romuleald
Copy link
Contributor

Peux-tu ajouter ajouter des cas d'usage dans le readme ?

@Aiso786
Copy link

Aiso786 commented Jan 31, 2022

Pas de Storybook sur ce composant ?

@MartinWeb MartinWeb closed this Jan 31, 2022
@MartinWeb MartinWeb reopened this Jan 31, 2022
@MartinWeb
Copy link
Contributor Author

Peux-tu ajouter ajouter des cas d'usage dans le readme ?

@romuleald Le fichier readme a été modifié avec un cas d'usage de la modification. Attends-tu quelque chose de plus ?

@MartinWeb
Copy link
Contributor Author

Pas de Storybook sur ce composant ?

@Aiso786 j'ai modifié le Storybook du composant pour ajouter un cas d'usage de l'utilisation du ReactNode en value. Tu peux le voir dans les fichiers modifiés. N'hésite pas à me dire si tu souhaiterais quelque chose de plus.

@Aiso786
Copy link

Aiso786 commented Jan 31, 2022

Pas de Storybook sur ce composant ?

@Aiso786 j'ai modifié le Storybook du composant pour ajouter un cas d'usage de l'utilisation du ReactNode en value. Tu peux le voir dans les fichiers modifiés. N'hésite pas à me dire si tu souhaiterais quelque chose de plus.

Ah oui, c'est bon, désolé je n'avais pas bien regardé.

Copy link
Contributor

@arnaudforaison arnaudforaison left a comment

Choose a reason for hiding this comment

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

Je pense qu'il y a un débat à avoir pour savoir si on garde encore value ET values

@MartinWeb MartinWeb force-pushed the feature/reactNodeForRestitutionValue branch from 77cbb3d to 05f8e2e Compare February 23, 2022 14:30
@MartinWeb
Copy link
Contributor Author

@arnaudforaison nous avons fait le point avec @samuel-gomez, on conserve value et values pour le moment et on refactorera cette partie dans un second temps. Je loggue l'issue sur ce sujet une fois la PR validée.

@samuel-gomez-axa samuel-gomez-axa merged commit 932f8a5 into AxaFrance:master Feb 24, 2022
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.

5 participants