Skip to content
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

rework MainIndexService assumptions #615

Open
julianseeger opened this issue Sep 27, 2016 · 3 comments
Open

rework MainIndexService assumptions #615

julianseeger opened this issue Sep 27, 2016 · 3 comments
Assignees
Labels

Comments

@julianseeger
Copy link
Collaborator

The MainIndexService makes some assumptions that don't make sense from the clients perspective, those should be checked and fixed.

For example:
Removing the identity is only possible with an email or phonenumber that is neither null nor blank. So I can't delete a previously uploaded identity when I previously deleted those fields ... but a user that no longer wants to be found would probably do exactly that and thus, could never delete his identity using the MainIndexService.
The scenarios get much worse if the identity gets deleted and re-imported...

@enkore
Copy link
Collaborator

enkore commented Sep 27, 2016

Yeah, it would be easier if we just allowed to say "remove everything you have on this identity", without being required to specify exactly what. (This is server side.)

@thechauffeur thechauffeur added this to the after RC milestone Sep 29, 2016
@jmt85 jmt85 modified the milestones: KW42/43, after RC Oct 17, 2016
@jmt85 jmt85 removed the po label Oct 17, 2016
@enkore
Copy link
Collaborator

enkore commented Oct 17, 2016

@thechauffeur thechauffeur modified the milestones: KW44/45, KW42/43 Oct 31, 2016
@thechauffeur
Copy link
Member

Let us talk about this next time.

@jmt85 jmt85 removed this from the KW44/45 milestone Nov 14, 2016
@enkore enkore mentioned this issue Nov 15, 2016
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants