Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Import
Contact
s from phone's contact book (#952) #966base: main
Are you sure you want to change the base?
Import
Contact
s from phone's contact book (#952) #966Changes from 15 commits
31d7111
254ddee
615f5fc
fb7400e
6342fa5
ff05247
94f23ba
ab95de7
fe6dd83
258e4ee
a018f01
0d479bf
fab51a3
52bb307
6a1e423
039a365
3c42204
d6d7d31
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Хм, кстати, а что будет, если контакты уже были импортированы, а мы заново этих же контактов импортируем? И, может, эту штуку в
ApplicationSettings
затолкать, чтобы оно жило дольше, чем текущая сессия? У нас тамintroduction
и лежит уже.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SleepySquash
Тогда у нас все контакты продублируются. Но я пока не вижу как мы можем от этого защититься с нашей стороны, так как любой индикатор который мы будем хранить локально может быть удален при очистке кеша, либо при удалении приложения. Думаю тут со стороны бекенда нужно менять логику создания контактов, чтобы у нас не могли добавляться несколько контактов с одинаковыми номерами/емейлами.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@krida2000, да, логику однозначно нужно корректировать там. Вот отсюда у меня и была просьба оценить и исследовать сначала, насколько при текущей концепции контактов вообще возможны все приседания, которые мы хотим сделать. Мне кажется, что это очень даже blocker тут.
Идеи пока две в таком случае:
И я пока придерживаюсь второго варианта, чтобы в prod'е не было недоделок.