Skip to content

added changes to NXD 13#27

Merged
taljacob2 merged 2 commits intomasterfrom
NXD-13-add-changes
Aug 19, 2025
Merged

added changes to NXD 13#27
taljacob2 merged 2 commits intomasterfrom
NXD-13-add-changes

Conversation

@Lina0Elman
Copy link
Contributor

No description provided.

@Lina0Elman Lina0Elman requested a review from LiavTB June 7, 2025 09:06
@Lina0Elman Lina0Elman changed the title added get resume by owner & added filename to resume model added changes to NXD 13 Jun 7, 2025
Copy link
Member

@taljacob2 taljacob2 left a comment

Choose a reason for hiding this comment

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

  1. Can you explain shortly what is done in this PR (and #26 PR)? I do not follow.

  2. I get these error messages when running the backend, because I did not upload a resume.

    Can we mute these errors?

    Error retrieving resume: Error: No resume found for user 67f926a44b84144eba9e1640
        at getResumeByOwner (I:\Tal\Code\js\NextStep\nextstep-backend\src\services\resume_service.ts:409:19)
        at processTicksAndRejections (node:internal/process/task_queues:105:5)
        at async Object.getResume (I:\Tal\Code\js\NextStep\nextstep-backend\src\controllers\resume_controller.ts:156:24)
    Error retrieving resume: Error: No resume found for user 67f926a44b84144eba9e1640
        at getResumeByOwner (I:\Tal\Code\js\NextStep\nextstep-backend\src\services\resume_service.ts:409:19)
        at processTicksAndRejections (node:internal/process/task_queues:105:5)
        at async Object.getResume (I:\Tal\Code\js\NextStep\nextstep-backend\src\controllers\resume_controller.ts:156:24)
    Error retrieving resume: Error: No resume found for user 67f926a44b84144eba9e1640
        at getResumeByOwner (I:\Tal\Code\js\NextStep\nextstep-backend\src\services\resume_service.ts:409:19)
        at processTicksAndRejections (node:internal/process/task_queues:105:5)
        at async Object.getResume (I:\Tal\Code\js\NextStep\nextstep-backend\src\controllers\resume_controller.ts:156:24)
    Error retrieving resume: Error: No resume found for user 67f926a44b84144eba9e1640
        at getResumeByOwner (I:\Tal\Code\js\NextStep\nextstep-backend\src\services\resume_service.ts:409:19)
        at processTicksAndRejections (node:internal/process/task_queues:105:5)
        at async Object.getResume (I:\Tal\Code\js\NextStep\nextstep-backend\src\controllers\resume_controller.ts:156:24)
    
  3. Are the new API routes documented in the swagger.yaml?

Base automatically changed from NXD-13 to master August 19, 2025 14:43
@taljacob2 taljacob2 merged commit c6b26c4 into master Aug 19, 2025
4 checks passed
@taljacob2 taljacob2 deleted the NXD-13-add-changes branch August 19, 2025 14:43
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.

2 participants