feat(FEC-13652): add meaningful error message #760
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the Changes
inside catch block of
loadMedia
, we classify errors that might come from BE (through provider) as Player errors. However, we would like to recognize them and provide thecategory
number that suits them.changes:
loadMedia
, before firing the error asPlayer
category, try to get the category according to the error payload.2
, it means it isService
category that came fromprovider
2001
is block action errorerror.code = 2001
we need to distinguish betweenSESSION_RESTRICTED
andCOUNTRY_RESTRICTED
2002
is media is not ready errorPlayer
category as defaultResolves FEC-13652