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

[EN-7791] fix(CV-PDF): left column width and display sections #409

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

guillobits
Copy link
Contributor

No description provided.

@guillobits guillobits added the bug Something isn't working label Dec 18, 2024
@guillobits guillobits requested a review from DDZBX December 18, 2024 10:13
@guillobits guillobits self-assigned this Dec 18, 2024
@guillobits guillobits temporarily deployed to entourage-job-front December 18, 2024 10:13 — with GitHub Actions Inactive
Copy link

cypress bot commented Dec 18, 2024

Default Project    Run #1571

Run Properties:  status check passed Passed #1571  •  git commit a517dfca56 ℹ️: Merge 767cac62c510a7db050b91a7ab6b0cd0c3b343d6 into 18f49f350a9c08776d58d411e270...
Project Default Project
Branch Review fix/EN-7791-cv-leftCol-width
Run status status check passed Passed #1571
Run duration 06m 21s
Commit git commit a517dfca56 ℹ️: Merge 767cac62c510a7db050b91a7ab6b0cd0c3b343d6 into 18f49f350a9c08776d58d411e270...
Committer CAUCHOIS Guillaume
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 42
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

Comment on lines -126 to 129
firstPageExperiences: cv.experiences.slice(0, nbOfExperienceFirstPage),
firstPageExperiences: cv.experiences, // all the xp on first page
firstPageFormations: cv.formations.slice(0, nbOfFormationFirstPage),
secondPageExperiences: cv.formations.slice(nbOfExperienceFirstPage),
secondPageExperiences: [], // no xp on second page
secondPageFormations: cv.formations.slice(nbOfFormationFirstPage),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Est ce que c'est possible que les experiences prennent plus d'une page ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Non dans tous les cas on peut afficher 5 éléments et on limite le nombre d'expérience à 5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants