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

Better reflect current SOP with not activated ID #25

Closed
wants to merge 1 commit into from

Conversation

melkor0
Copy link

@melkor0 melkor0 commented Apr 14, 2020

This PR changes the message to the user what he should do when he is using a not online-activated ID.
Example: An ID is issued and the online function is not activated. The responsible regsitry office (Einwohnermeldeamt) is called A.
The user moves and another is now another registry office (B) is responsible for the user. The user wishes to user AusweisApp2 to authenticate.
In the current state the message can be understood to contact registry office A, which in this case is not responsible for the user.

This PR changes the message to the user what he should do when he is using a not online-activated ID.
Example: An ID is issued and the online function is not activated. The responsible regsitry office (Einwohnermeldeamt) is called A.
The user moves and another is now another registry office (B) is responsible for the user. The user wishes to user AusweisApp2 to authenticate.
In the current state the message can be understood to contact registry office A, which in this case is not responsible for the user.
@CLAassistant
Copy link

CLAassistant commented Apr 14, 2020

CLA assistant check
All committers have signed the CLA.

@ghost
Copy link

ghost commented Apr 14, 2020

Congratulations 🎉. DeepCode analyzed your code in 1.114 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard

@misery
Copy link
Contributor

misery commented Jun 26, 2020

Thanks for your feedback.

We are working with usability experts and those strings are agreed with the BSI. We already started a new discussion about this now and will change this in a arranged manner. Thanks for that. We will use this as example and proposal.

I think this could be changed in next minor release.

@misery misery closed this Jun 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants