feat(api): add emergency contacts database functions #1081
Conversation
| } | ||
|
|
||
| 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.
Просто в рамках логики апи это разные штуки. Это в рамках логики фронта надо запросы по очереди отправлять
| type HeartbeatType string | ||
|
|
||
| const ( | ||
| HearbeatTypeNotSet HeartbeatType = "type_not_set" |
There was a problem hiding this comment.
| HearbeatTypeNotSet HeartbeatType = "type_not_set" | |
| HeartbeatTypeNotSet HeartbeatType = "type_not_set" |
?
| // IsValid checks if such an heartbeat type exists. | ||
| func (heartbeatType HeartbeatType) IsValid() bool { | ||
| switch heartbeatType { | ||
| case HeartbeatNotifierOff: |
There was a problem hiding this comment.
А почему решил сюда не добавлять type_not_set?
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