Skip to content

Conversation

@gavlyukovskiy
Copy link
Collaborator

No description provided.

Copy link
Contributor

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 introduces a deployment configuration for running the application on a VPS using Docker Compose, replacing the previous Digital Ocean App Platform deployment approach. The changes update port configurations and add a complete deployment setup with Nginx as a reverse proxy.

  • Removes Digital Ocean-specific deployment workflows and configurations
  • Adds a new deployment directory with Docker Compose setup including Nginx reverse proxy, automatic updates via Watchtower
  • Updates port mappings from 80 to 3000 for the frontend across development and deployment configurations

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
frontend/Dockerfile Updates exposed port from 80 to 3000
docker-compose.yaml Updates Postgres to version 18, changes frontend port mapping to 3000, removes backend service
deployment/nginx.conf Adds Nginx configuration for reverse proxy routing
deployment/docker-compose.yaml Adds complete deployment setup with all services
deployment/README.md Adds deployment instructions for VPS setup
README.md Updates documentation to reflect new port and deployment approach
.github/workflows/on-release-tag.yml Removes automated deployment step
.github/workflows/deploy-to-digital-ocean.yml Removes entire Digital Ocean deployment workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@gavlyukovskiy gavlyukovskiy merged commit 768906b into main Oct 19, 2025
@Breus Breus deleted the vps-deployment branch October 21, 2025 07:03
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.

3 participants