-
Notifications
You must be signed in to change notification settings - Fork 209
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
[ESD-30932] Improve error handling #720
Conversation
Hi, thanks for looking at this. I notice this seems to only be for Android. Is there more code or another PR coming that forwards the api error code and description on iOS devices as well? |
@rick-lannan-upgrowth agree with you on that. I have pushed more changes to include backend error codes for iOS as well. Can you take a look? |
Hi @poovamraj, looks good. It seems that iOS and Android seem to be returning the root cause api error code and description now. I look forward to testing this one |
@Widcket Can you review the iOS part of this PR? @auth0/dx-sdks-engineer Anyone up for reviewing this PR? |
Manual testing results Before
Android
After
Android
|
@poovamraj @Widcket any updates? |
@rick-lannan-upgrowth as you can see we are in the process of reviewing the PR. @Widcket will verify the iOS changes and we can merge it and then continue with the release. |
Hi again, just wondering when the release is coming out? Thanks |
Hi @rick-lannan-upgrowth we have released the new version. Do check it out and let us know your feedback. |
Changes
References
Testing
We have done manual testing to ensure the error details are propagated