Skip to content

Feature/cv management#3

Open
jeanbfly wants to merge 14 commits intodevelopfrom
feature/cv-management
Open

Feature/cv management#3
jeanbfly wants to merge 14 commits intodevelopfrom
feature/cv-management

Conversation

@jeanbfly
Copy link
Collaborator

@jeanbfly jeanbfly commented Mar 13, 2026

Description

This PR initializes the project and adds CV management features.

Users can now:

  • Create a CV
  • Open a CV
  • Rename a CV
  • Delete a CV
  • Duplicate a CV

Scope

This PR focuses only on the initial CV management features.
Additional improvements (tests, refactoring) can be done in follow-up PRs.

Changes

  • Added backend routes for CV management
  • Added frontend views
  • Implemented frontend logic for CV operations

Known limitations

  • Some parts are missing tests
  • Some parts of the code could be improved for readability (first contribution to the project)
  • Some parts of the code could be refactor and improved

How to test

  1. Start the containers
  2. Refresh the database schema
  3. Run the frontend and backend (front:with-back)
  4. Navigate to the CV management section and test the create, rename, duplicate, and delete actions
  5. Code is located respectively in @libs/users-backend and @libs/users-front as requested

Notes

This PR is relatively large because it introduces the initial CV management features.

Feedback is welcome.

@AmauryD AmauryD requested a review from Copilot March 13, 2026 10:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 286 out of 341 changed files in this pull request and generated no comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copy link
Member

@remadex remadex left a comment

Choose a reason for hiding this comment

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

Tu as actuellement mis toutes tes modifications sur les CV dans les libs users/.
Je sais que vu la taille de ton projet, il n'est pas nécessaire de passer par les libs mais dans ce cas-là ne met pas tout dans users, l'idée serait de défaire la notion de libs et de tout mettre dans @apps/

En attendant, tu mélanges les fonctionnalités users et curriculum dans la libs users et ça ne va pas...

2 choix possibles, soit tu mets tout dans les apps, soit tu fais une libs pour curriculum.

Et enfin, attention à ne pas trop te reposer sur l'IA pour tes différents développements 😬
On est des mauvais exemples, mais on sait ce qu'on fait 😜

@jeanbfly
Copy link
Collaborator Author

jeanbfly commented Mar 16, 2026

Tu as actuellement mis toutes tes modifications sur les CV dans les libs users/. Je sais que vu la taille de ton projet, il n'est pas nécessaire de passer par les libs mais dans ce cas-là ne met pas tout dans users, l'idée serait de défaire la notion de libs et de tout mettre dans @apps/ En attendant, tu mélanges les fonctionnalités users et curriculum dans la libs users et ça ne va pas...

2 choix possibles, soit tu mets tout dans les apps, soit tu fais une libs pour curriculum.

Et enfin, attention à ne pas trop te reposer sur l'IA pour tes différents développements 😬 On est des mauvais exemples, mais on sait ce qu'on fait 😜

voilà, je viens de rajouter 2 commits de fix qui couvre les problèmes. Certaines parties ont été adaptées, d'autres corrigées. Il manque encore quelques tests pour être parfait. Pourrais-tu review mes corrections ? @remadex
PS: j'utilise pas l'IA pour l'UI, je n'ai juste pas encore le niveau 😂

@jeanbfly jeanbfly requested a review from remadex March 16, 2026 16:32
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.

4 participants