feat(api): add emergency contacts database functions #1081
Conversation
datatypes.go
Outdated
| @@ -222,6 +222,33 @@ func (contact *ContactData) ToTemplateContact() *templating.Contact { | |||
| } | |||
| } | |||
|
|
|||
| // EmergencyContactType are Moira's special internal types of problems. | |||
There was a problem hiding this comment.
есть предложение, прекратить это дело плодить, завести папку
обозвать дататайп (выберете лучшее название)
в этом пр все, что касается эмергенси контанкта унести туда
отдельно раздербанить этот жирный файл на кучу файлов в папке
There was a problem hiding this comment.
Мне кажется, имеет смысл вынести это в отдельный PR, так как если я делаю папку datatypes и добавлю туда файлы datatypes.go и emergency_contacts.go, то ломается полпроекта, тк все зависит от datatypes.go, а если добавить в папку только emergency_contacts.go, то разъедется логика, в разных местах будет папка и файл, что странно
There was a problem hiding this comment.
нет, сделай в этом папку и занеси туда emergency_contacts.go
и следующий пр - с переносом только файла в папку просто
и отдельно задачу на распил
план такой
| } | ||
|
|
||
| emergencyContact, getEmergencyContactErr := connector.GetEmergencyContact(contactID) | ||
| if getEmergencyContactErr != nil && !errors.Is(getEmergencyContactErr, database.ErrNil) { |
There was a problem hiding this comment.
Если есть эмёрдженси контакт -- мы удаляем и его? Может логика должна быть наоборот, как с подписками и контактами: сначала удалите все зависимости, а потом только удаляйте контакты
There was a problem hiding this comment.
Вопрос обсуждаемый и зависит от того, как это будет выглядеть в UI, если это будет галка внутри UI заполнения контакта, то логично было бы не просить пользователя отжимать галку, а просто удалять и то, и то. А если в UI будет отдельное окошко для этих контактов, то вариант с тем, чтобы пользователь сначала убирал эти контакты, а затем удалял основной имеет смысл быть. Но, честно, с учетом того, что связь 1-1, то удалять для меня выглядит более логичным вариантом
There was a problem hiding this comment.
Вынесу в общее обсуждение
There was a problem hiding this comment.
Просто в рамках логики апи это разные штуки. Это в рамках логики фронта надо запросы по очереди отправлять
Add emergency contacts database functions
Added database functions for emergency contacts - these are the contacts to which notifications will go to users in case of emergency types of problems in Moira