merge staging to prod#41
Merged
Merged
Conversation
… automated build, push, and deployment of the API to Cloud Run upon pushes to the staging branch
… environment variables for project ID, service name, region, and Docker registry; add .env.staging to .gitignore
…riable and adjust Docker image paths for consistency in production and staging configurations
Feat/new messages
Feat/new messages
Feat/new messages
🐛 fix phoneNumber
…e.json and pnpm-lock.yaml
…and update RidersService to use PipelineStage for aggregation
…rch pipeline, and update SearchModule with new service and DTO
…r input validation in NestJS, enhancing error handling with BadRequestException.
…r for handling contact email submissions, including validation using Zod and integration with nestjs-resend for email delivery.
…de 'toEmail' field, and update ContactController to send personalized emails with user input for recipient and message content.
…ethod to use 'send-one' endpoint for improved clarity and consistency in API design.
…r managing sponsor messages, update ContactController to save messages upon email sending, and refine ContactEmailDto structure for improved data handling.
…tion for async loading, update AppService greeting message, add contracts route in FeaturesModule, and modify ContactController to use dynamic email subject from user input.
…tant in MongoDBConnection, enhance ContactController to log user data and handle sponsor messages, and update AuthenticationGuard to check method and class security metadata.
…sending sponsor messages, ensuring proper status updates in the database, and refactor getSponsorMessages method to utilize authenticated user data.
…tion for the Contact API, detailing authentication, available routes, data schemas, error codes, and usage examples for sending emails and retrieving sponsor messages.
… and findBySlug methods to streamline data retrieval.
Fix unacessible public portfolio
Contributor
There was a problem hiding this comment.
Pull Request Overview
This is a significant merge from staging to deploy, introducing a comprehensive search functionality with advanced filtering, file upload capabilities through Google Cloud Storage integration, enhanced authentication features, and improved contract management system.
- Search system with complex MongoDB aggregation pipelines supporting advanced filtering (sports, location, age, availability)
- Google Cloud Storage service for file uploads with multipart support for avatars and images
- Enhanced authentication with optional authentication decorator and improved contract messaging system
Reviewed Changes
Copilot reviewed 31 out of 34 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
| src/shared/gcp/services/storage.service.ts | New GCP Storage service for file upload/deletion operations |
| src/features/search/ | Complete search module with advanced filtering, saved searches, and MongoDB aggregation |
| src/features/riders/services/riders.service.ts | Enhanced with file upload methods and improved aggregation pipeline typing |
| src/features/contracts/services/contracts.service.ts | Updated contract schema and messaging system with read status tracking |
| src/features/auth/ | Improved authentication with optional auth decorator and guard enhancements |
| src/main.ts | Added multipart support and improved CORS configuration |
| package.json | Added GCP Storage, multipart handling, and Zod validation dependencies |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
… ContactController and SponsorMessageService to use Logger for better error tracking.
add resend module and contact feature
…r improved logging and error handling
…e for consistency and clarity
ImJustLucas
approved these changes
Jul 21, 2025
Lucas-Huerta
approved these changes
Jul 21, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.