#169 GetFileStorageParamsUseCase #22
Conversation
| @StringField({ | ||
| label: 'Тип файла', | ||
| nullable: true, | ||
| }) | ||
| fileType: string; |
There was a problem hiding this comment.
Миграции для модели File у нас создаются же в проектах? Тогда получается что нужно в MigrationGuide написать чтобы запустили генерацию миграций после накатывания этого изменения.
There was a problem hiding this comment.
Напишу, только MigrationGuide во время выпуска новой версии заполняется обычно. Или предлагаешь в рамках PR сразу набрасывать наброски?
There was a problem hiding this comment.
В целом он только с выпуском новой версии нужен, но тогда надо как-то так сделать, чтобы это не забылось во время проверки новой версии...
There was a problem hiding this comment.
Ну то есть добавить набросок в MigrationGuide? В отдельный раздел какой-нибудь типа Pending
There was a problem hiding this comment.
Или может создать задачу, в которой прописать что нужно обновить версию и добавить конкретную инфу в MigrationGuide?
There was a problem hiding this comment.
Я новые версии без задач оформляю обычно, прохожу по истории коммитов и всё записываю в описание версии и MigrationGuide
There was a problem hiding this comment.
Получается, тогда в коммите это стоит отметить? Если по-правильному, то через BREAKING CHANGE: ...
There was a problem hiding this comment.
Оно не особо то и BREAKING 🌚
А так я планировал сразу новую версию и выкатить, мне на проекте это обновление нужно. Так что не забуду
There was a problem hiding this comment.
Оно не особо то и BREAKING 🌚
Да, но тогда надо придумать такие изменения, которые по факту не будут работать без дополнительных действий.
А так я планировал сразу новую версию и выкатить, мне на проекте это обновление нужно. Так что не забуду
Я бы отталкивался от процессов, а не от конкретных людей 🌚. Так что если не сложно, то зафиксируй это, хотя бы в задаче )
There was a problem hiding this comment.
зафиксируй это, хотя бы в задаче )
оставил в задаче комментарий
src/usecases/getFileStorageParams/interfaces/IGetFileStorageParamsUseCase.ts
Show resolved
Hide resolved
3 participants
No description provided.