Изменение модели и миграции для полей настроек уведомлений

gorskyolga commented 3 weeks ago

Что сделано

Изменила модель и миграцию, как указано в задаче https://github.com/Studio-Yandex-Practicum/ProCharity_back_2.0/issues/654. Чтобы не делать downgrade существующей миграции на сервере, создала две миграции: одна - для удаления столбцов с настройками, вторая - для создания столбцов.

Как проверяла

Локально. Применила миграции. С помощью DBeaver проверила, что в таблице external_site_users новые поля имеют тип bool и значение NULL. Запустила бот и создала нового пользователя сайта через эндпоинт для их регистрации, пользователь успешно создался.

Примечание

После деплоя на тестовый сервер и применения там миграций две промежуточные миграции будут удалены (с исправлением revision id). Предварительно протестировала локально, что после такого изменения downgrade и upgrade миграций работают корректно.

eugemos commented 3 weeks ago

К стати, идея с удалением промежуточных миграций не очень хорошая. Ты ориентируешься на то, что на сервере проблем не будет, но что будут делать те члены команды, у которых к тому времени уже будут применены те миграции, которые ты хочешь удалить?

gorskyolga commented 3 weeks ago

К стати, идея с удалением промежуточных миграций не очень хорошая. Ты ориентируешься на то, что на сервере проблем не будет, но что будут делать те члены команды, у которых к тому времени уже будут применены те миграции, которые ты хочешь удалить?

Я подожду с удалением, пока все не накатят новую миграцию.

Studio-Yandex-Practicum / ProCharity_back_2.0