Skip to content

fix: Rename id to UUID#56

Merged
vbetsch merged 9 commits intodevelopfrom
refactor/rename-id-to-uuid
Jul 31, 2025
Merged

fix: Rename id to UUID#56
vbetsch merged 9 commits intodevelopfrom
refactor/rename-id-to-uuid

Conversation

@vbetsch
Copy link
Copy Markdown
Owner

@vbetsch vbetsch commented Jul 31, 2025

No description provided.

@vbetsch vbetsch requested a review from Copilot July 31, 2025 17:46
@vbetsch vbetsch self-assigned this Jul 31, 2025
@vbetsch vbetsch added database About database schema or models api About endpoints and services Code refactoring Code refactoring (for the release changelog) release:fix Bug fixes (for the release changelog) labels Jul 31, 2025
Copy link
Copy Markdown
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 renames the primary key field from id to uuid in the Vault model to better reflect its data type and purpose. The change maintains backward compatibility by mapping the uuid field to id in the DTO layer.

  • Database schema migration to rename the primary key column from id to uuid
  • Updated repository method parameter names to use uuid instead of id
  • Added DTO mapping to maintain id field in API responses while using uuid internally

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
prisma/schema.prisma Changes primary key field name from id to uuid in Vault model
prisma/migrations/20250731173933_id_to_uuid/migration.sql Auto-generated migration script for the schema change
src/modules/api/repositories/vaults.repository.ts Updates delete method parameter from id to uuid
src/modules/api/adapters/vault.adapter.ts Adds id field mapping from uuid in DTO transformation
package.json Adds new Prisma-related npm scripts for migrations
Makefile Enhances migration commands with proper naming and adds database reset functionality

@vbetsch vbetsch merged commit e01ba9c into develop Jul 31, 2025
4 checks passed
@vbetsch vbetsch deleted the refactor/rename-id-to-uuid branch July 31, 2025 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api About endpoints and services Code refactoring Code refactoring (for the release changelog) database About database schema or models release:fix Bug fixes (for the release changelog)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants