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.

Sign up for GitHub

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

Jump to bottom

Alert when entering wrong credentials should be more user friendly #10

Closed
boetter opened this issue Jan 5, 2016 · 7 comments
Closed

Alert when entering wrong credentials should be more user friendly #10

boetter opened this issue Jan 5, 2016 · 7 comments

Comments

@boetter
Copy link
Contributor

boetter commented Jan 5, 2016

Used to work just fine, but in the latest version I get this error when trying to add my account:

screen shot 2016-01-05 at 10 38 00
@olegam
Copy link
Owner

olegam commented Jan 5, 2016

Works for me. Are you sure both codes are correct and you didn't swap them?

@watson
Copy link

watson commented Jan 5, 2016

I got the same error when I accidentally entered the wrong CPR number

@olegam
Copy link
Owner

olegam commented Jan 5, 2016

The text should be changed to something more friendly. Explaining that the credentials could not be validated.

@olegam olegam changed the title Can't sign in Alert when entering wrong credentials should be more user friendly Jan 12, 2016
@olegam olegam closed this as completed in bf8126c Jan 20, 2016
@olegam
Copy link
Owner

olegam commented Jan 20, 2016

@sebreh could you help with a Swedish translation here?

Postboks/Postboks/sv.lproj/Localizable.strings

Line 33 in bf8126c

"invalid-login-credentials" = "The credentials you entered could not be verified.";
?

@sebreh sebreh mentioned this issue Jan 20, 2016
Merged
@boetter
Copy link
Contributor Author

boetter commented Jan 31, 2016

I figured out why it gave me an error!

When entering social security number I entered XXXXX-XXXX, it accepted when I did it as XXXXXXXXXX

@olegam
Copy link
Owner

olegam commented Jan 31, 2016

Ahh right. We should remove non-numeric characters from the input.

@olegam olegam reopened this Jan 31, 2016
@olegam olegam closed this as completed Jan 31, 2016
@olegam olegam mentioned this issue Jan 31, 2016
Open
@olegam
Copy link
Owner

olegam commented Jan 31, 2016

I created a new issue for this: #27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@watson @olegam @boetter