Skip to content

Conversation

@stasel
Copy link
Member

@stasel stasel commented Feb 2, 2026

  • Update all packages to the latest version
  • Fixed TS errors with missing casting on express query params

Copilot AI review requested due to automatic review settings February 2, 2026 19:55
@HackYourFutures HackYourFutures temporarily deployed to dojo-backend-update-pac-dppxaf February 2, 2026 19:56 Inactive
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.

Pull request overview

This PR upgrades backend packages, particularly addressing compatibility issues with AWS SDK v3. The changes include updating dependency versions in package.json and adding explicit String() conversions for request parameters throughout the codebase.

Changes:

  • Upgraded multiple packages including AWS SDK, Sentry, Mongoose, and various dev dependencies
  • Added String() wrapper to all req.params accesses to ensure type safety
  • Modified controller methods across User, Trainee, Test, Strike, Interaction, and other related modules

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
server/package.json Updated versions for AWS SDK, Sentry, Mongoose, faker, dotenv, and various @types packages
server/src/controllers/UserController.ts Wrapped userId extraction with String() in updateUser and deleteUser methods
server/src/controllers/Trainee/TraineeController.ts Added String() conversion for trainee ID parameters
server/src/controllers/Trainee/TestController.ts Applied String() conversion to trainee and test ID parameters across multiple methods
server/src/controllers/Trainee/StrikeController.ts Wrapped trainee and strike ID parameters with String()
server/src/controllers/Trainee/ProfilePictureController.ts Added String() conversion for trainee ID in profile picture operations
server/src/controllers/Trainee/LetterController.ts Wrapped trainee ID with String() in letter generation
server/src/controllers/Trainee/InteractionController.ts Applied String() conversion to trainee and interaction ID parameters
server/src/controllers/Trainee/EmploymentHistoryController.ts Added String() wrapper for trainee and employment history ID parameters

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

@stasel stasel added the BE Backend ticket label Feb 2, 2026
@stasel stasel added this to Dojo Feb 2, 2026
@stasel stasel requested a review from Copilot February 2, 2026 22:07
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.

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated no new comments.


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

@stasel stasel force-pushed the backend/update-packages branch from b955e2f to 652e493 Compare February 3, 2026 09:18
@stasel stasel changed the title WIP: Upgrade backend packages Upgrade backend packages Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BE Backend ticket

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants