Skip to content
This repository was archived by the owner on Jul 6, 2025. It is now read-only.

Conversation

@gustaavik
Copy link
Member

This pull request introduces several changes across multiple areas of the codebase, including environment configuration, authentication, user management, and updates to order and product handling. Below is a summary of the most important changes grouped by theme:

Environment and Configuration Updates:

  • Removed unused Stripe-related environment variables from deploy/.env.example and added a FRONTEND_HOST variable. [1] [2]
  • Updated deploy/docker-compose.yml to adjust MySQL port mappings and replace environment variable references for database credentials with hardcoded values. [1] [2]
  • Added FRONTEND_URL as an environment variable in deploy/docker-compose.yml.

Authentication Enhancements:

  • Added a new /me endpoint in AuthController to retrieve the current user's profile using a validated access token.
  • Refactored AuthResponse and NextAuthResponse to include UserProfileResponse instead of individual user fields like id, name, and email. [1] [2] [3]
  • Updated AuthenticationResult to include a User object alongside the AuthenticatedUser. [1] [2] [3]

User Management:

  • Introduced a new UserController with endpoints to fetch all users (with pagination) and retrieve a user by ID, accessible only to admins.

Order and Product Enhancements:

  • Added updatedAt timestamp fields to OrderDetailsResponse and OrderDetailsDTO. [1] [2] [3]
  • Extended UpdateProductRequest and ProductSummaryResponse to include imageUrl and isActive fields for better product management. [1] [2] [3] [4]

Miscellaneous:

  • Removed unused roles (ROLE_MANAGER) from the UserRole enum.
  • Added a placeholder method canCaptureOrder in OrderApplicationService for future order capture functionality.

@gustaavik gustaavik merged commit 12f9f74 into main May 8, 2025
1 check passed
@gustaavik gustaavik deleted the patch-bug-fixes branch May 8, 2025 22:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants