-
Notifications
You must be signed in to change notification settings - Fork 6
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
Handle more kratos error codes #280
Conversation
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.
would be good to eventually get a test for the ui errors
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.
I am approving this to move forward. BUT as I mentioned in the issue as well, I think we will need to go through all the Kratos errors and create a (nearly) exhaustive mapping. Adding an error code every time one pops up does not scale very well.
7437353
e2b9b83
to
7437353
Compare
I went through the list of codes. Some of them are already mitigated by validation done on frontend (e.g. password requirements), others make no value to the end user. |
fixes #276