Import Contacts from phone's contact book (#952)

[krida2000]

Resolves #952

Synopsis

Необходимо реализовать перенос контактов из адресной книги пользователя в приложение.

Solution

Перенос контактов будет реализован.

Checklist

• Created PR:
  • In draft mode
  • Name contains issue reference
  • Has type and k:: labels applied
• Before review:
  • Documentation is updated (if required)
  • Tests are updated (if required)
  • Changes conform code style
  • CHANGELOG entry is added (if required)
  • FCM (final commit message) is posted or updated
  • Draft mode is removed
• Review is completed and changes are approved
  • FCM (final commit message) is approved
• Before merge:
  • Milestone is set
  • PR's name and description are correct and up-to-date
  • All temporary labels are removed

[krida2000]

FCM

Use `fast_contacts` to import device's contact book (#966, #952)

[SleepySquash]

@krida2000, пока не уверен по поводу того, куда класть код, обращающийся напрямую к списку контактов устройства. Точно не в сторе, но в сервис/воркер или в контроллер - это вопрос. Скорее в сервис/воркер, я думаю, т.к. мы можем захотеть откуда-нибудь из страницы профиля по итогу дёргать обновление синхронизации. Но тогда вопрос - "сервис или воркер". С локализацией эта штука не работает, однако с устройством работает. Но и уведомления по факту с устройством работает.

[SleepySquash]

Хм, кстати, а что будет, если контакты уже были импортированы, а мы заново этих же контактов импортируем? И, может, эту штуку в ApplicationSettings затолкать, чтобы оно жило дольше, чем текущая сессия? У нас там introduction и лежит уже.

[krida2000]

@SleepySquash

Хм, кстати, а что будет, если контакты уже были импортированы, а мы заново этих же контактов импортируем?

Тогда у нас все контакты продублируются. Но я пока не вижу как мы можем от этого защититься с нашей стороны, так как любой индикатор который мы будем хранить локально может быть удален при очистке кеша, либо при удалении приложения. Думаю тут со стороны бекенда нужно менять логику создания контактов, чтобы у нас не могли добавляться несколько контактов с одинаковыми номерами/емейлами.

[SleepySquash]

@krida2000, да, логику однозначно нужно корректировать там. Вот отсюда у меня и была просьба оценить и исследовать сначала, насколько при текущей концепции контактов вообще возможны все приседания, которые мы хотим сделать. Мне кажется, что это очень даже blocker тут.

Идеи пока две в таком случае:

1. Закомментировать отправку создания контактов на бэкэнд, пометив тудушкой.
2. Пометить PR как waiting:materials и ждать исправления этого момента.

И я пока придерживаюсь второго варианта, чтобы в prod'е не было недоделок.