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: 5271 indemnité de licenciement créer un questionnaire spécifique #5287

Merged

Conversation

Viczei
Copy link
Contributor

@Viczei Viczei commented Jul 25, 2023

No description provided.

@Viczei Viczei self-assigned this Jul 25, 2023
@Viczei Viczei linked an issue Jul 25, 2023 that may be closed by this pull request
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@Viczei Viczei temporarily deployed to review-auto July 25, 2023 08:51 — with GitHub Actions Inactive
@Viczei Viczei temporarily deployed to review-auto July 25, 2023 12:48 — with GitHub Actions Inactive
@Viczei Viczei temporarily deployed to review-auto July 25, 2023 13:54 — with GitHub Actions Inactive
@Viczei Viczei temporarily deployed to review-auto July 25, 2023 14:36 — with GitHub Actions Inactive
@Viczei Viczei temporarily deployed to review-auto July 25, 2023 15:09 — with GitHub Actions Inactive
@Viczei Viczei temporarily deployed to review-auto July 25, 2023 15:28 — with GitHub Actions Inactive
@maxgfr maxgfr marked this pull request as ready for review July 31, 2023 07:46
Copy link
Contributor

@m-maillot m-maillot left a comment

Choose a reason for hiding this comment

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

Il y a un bug quand on ferme le questionnaire dès le début.

Comment on lines +45 to +47
badEventValue={FEEDBACK_RESULT.NOT_AT_ALL}
averageEventValue={FEEDBACK_RESULT.AVERAGE}
goodEventValue={FEEDBACK_RESULT.YES}
Copy link
Contributor

Choose a reason for hiding this comment

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

On ne pourrait pas mettre des valeurs par défaut ici ?

@Viczei Viczei temporarily deployed to review-auto July 31, 2023 08:55 — with GitHub Actions Inactive
</Flex>
<RelatedItems items={relatedItems} />
<Feedback url={router.asPath} />
{router.asPath !== "/outils/indemnite-licenciement" && (
Copy link
Member

Choose a reason for hiding this comment

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

peut être faire un array pour éviter d'avoir une url statique dans le code en mode MY_ARRAY.includes(router.asPath)

…uestionnaireAdvanced.tsx

Co-authored-by: Martial Maillot <[email protected]>
…uestionnaireAdvanced.tsx

Co-authored-by: Martial Maillot <[email protected]>
…uestionnaireAdvanced.tsx

Co-authored-by: Martial Maillot <[email protected]>
…uestionnaireAdvanced.tsx

Co-authored-by: Martial Maillot <[email protected]>
…uestionnaireAdvanced.tsx

Co-authored-by: Martial Maillot <[email protected]>
@maxgfr
Copy link
Member

maxgfr commented Aug 3, 2023

@m-maillot en fait, il n'y avait pas de bug, c'est juste la zone de click de la croix qui est petite. du coup, ce que j'ai fait c'est que je l'ai agrandi

@maxgfr maxgfr merged commit 969c330 into dev Aug 3, 2023
7 of 8 checks passed
@maxgfr maxgfr deleted the 5271-indemnité-de-licenciement-créer-un-questionnaire-spécifique branch August 3, 2023 15:22
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.

[indemnité de licenciement] créer un questionnaire spécifique
5 participants