Added ErrorContext to give more context which action caused the actual error.#1530
Added ErrorContext to give more context which action caused the actual error.#1530pamapa merged 6 commits intoauthts:mainfrom blaubaer:error-context
ErrorContext to give more context which action caused the actual error.#1530Conversation
|
Looks great. I see that you added an additional property to avoid breaking the API. Extending the existing error property would make it easier for usage. Either you have an error object or its null. If you have one you can find the message + details... For me it would be ok in that situation to bump the major version and break the ABI... |
|
Ideally we also adapt error object within that new component: #1476 |
|
Sound like a good idea. Also since now |
|
I've just adjusted as discussed. I've tried to find a sweet spot between do not introduce a new property but also do not break the current API tooo much. Therefore: What was in the initial contribution |
|
Thanks for contributing! |
Checklist
Details
This added
AuthState.errorContext: ErrorContextwhich holds information what was causing the issue. Most likely which method and (if available) the providedargs.