fix: Fix the PostgreSQL enum and generate a new database migration.#35
Merged
Conversation
Contributor
Руководство для рецензентаОбновляет модель User в SQLAlchemy, чтобы определить именованный, не допускающий NULL PostgreSQL enum для поля role, и регенерирует начальную миграцию Alembic, чтобы согласовать схему базы данных (users, bans, magic_tokens, services, appointments) с текущими моделями, заменяя более старые миграции. Диаграмма отношений сущностей для обновлённой начальной схемы и enum роли пользователяerDiagram
users {
int id PK
enum role
}
bans {
int id PK
int user_id FK
int banned_by FK
int revoked_by FK
}
magic_tokens {
int id PK
int user_id FK
}
services {
int id PK
int entrepreneur_id FK
}
appointments {
int id PK
int service_id FK
int entrepreneur_id FK
int user_id FK
}
users ||--o{ bans : user_id
users ||--o{ bans : banned_by
users ||--o{ bans : revoked_by
users ||--o{ magic_tokens : user_id
users ||--o{ services : entrepreneur_id
users ||--o{ appointments : entrepreneur_id
users ||--o{ appointments : user_id
services ||--o{ appointments : service_id
Изменения на уровне файлов
Подсказки и командыВзаимодействие с Sourcery
Настройка работыОткройте свою панель управления, чтобы:
Получение помощи
Original review guide in EnglishReviewer's GuideUpdates the User SQLAlchemy model to define a named, non-nullable PostgreSQL enum for the role field and regenerates the initial Alembic migration to align the database schema (users, bans, magic_tokens, services, appointments) with the current models, replacing older migrations. Entity relationship diagram for updated initial schema and user role enumerDiagram
users {
int id PK
enum role
}
bans {
int id PK
int user_id FK
int banned_by FK
int revoked_by FK
}
magic_tokens {
int id PK
int user_id FK
}
services {
int id PK
int entrepreneur_id FK
}
appointments {
int id PK
int service_id FK
int entrepreneur_id FK
int user_id FK
}
users ||--o{ bans : user_id
users ||--o{ bans : banned_by
users ||--o{ bans : revoked_by
users ||--o{ magic_tokens : user_id
users ||--o{ services : entrepreneur_id
users ||--o{ appointments : entrepreneur_id
users ||--o{ appointments : user_id
services ||--o{ appointments : service_id
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Привет — я оставил несколько общих замечаний:
- В новой миграции
initial_schemaпуть понижения версии удаляет таблицуusers, но не удаляет соответствующий тип enum PostgreSQLuser_role, что может оставить «осиротевшие» типы в базе данных; рассмотрите возможность явного удаления этого enum-типа вdowngrade().
Подсказка для AI-агентов
Пожалуйста, исправьте замечания из этого code review:
## Общие комментарии
- В новой миграции `initial_schema` путь понижения версии удаляет таблицу `users`, но не удаляет соответствующий тип enum PostgreSQL `user_role`, что может оставить «осиротевшие» типы в базе данных; рассмотрите возможность явного удаления этого enum-типа в `downgrade()`.Sourcery бесплатен для open source — если вам нравятся наши обзоры, пожалуйста, подумайте о том, чтобы поделиться ими ✨
Original comment in English
Hey - I've left some high level feedback:
- In the new
initial_schemamigration, the downgrade path drops theuserstable but does not drop the underlying PostgreSQL enum typeuser_role, which can leave orphaned types in the database; consider explicitly dropping the enum type indowngrade().
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In the new `initial_schema` migration, the downgrade path drops the `users` table but does not drop the underlying PostgreSQL enum type `user_role`, which can leave orphaned types in the database; consider explicitly dropping the enum type in `downgrade()`.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Coverage Report for CI Build 26835238459Coverage remained the same at 30.025%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
1 similar comment
Coverage Report for CI Build 26835238459Coverage remained the same at 30.025%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
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
1 participant
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.
Summary by Sourcery
Обновлено определение enum для ролей пользователя и приведена схема базы данных в соответствие с текущими доменными моделями посредством новой начальной миграции.
Исправления ошибок:
Улучшения:
Original summary in English
Summary by Sourcery
Update the user role enum definition and align the database schema with the current domain models via a new initial migration.
Bug Fixes:
Enhancements: