The main focus of this was to support overriding missing scope errors. The typical errors returned by the BE aren't very useful, so this makes it easy for us to provide better messaging to the end user when they attempt to make a request that fails with a 403 status and MISSING_SCOPE category.
I removed all of the places where we were passing in projectName to the api error context because we don't need that anymore (I'm removing it here).
I also cleaned up a few things:
I removed some dead translations that weren't being used anywhere anymore
I slightly modified some code in overrideErrors to prevent a potential issue in the scenario where err.response.data.context doesn't exist
I removed some code in the sandbox sync command that was manually handling the missing scope error
Description and Context
The main focus of this was to support overriding missing scope errors. The typical errors returned by the BE aren't very useful, so this makes it easy for us to provide better messaging to the end user when they attempt to make a request that fails with a 403 status and
MISSING_SCOPEcategory.I removed all of the places where we were passing in
projectNameto the api error context because we don't need that anymore (I'm removing it here).I also cleaned up a few things:
err.response.data.contextdoesn't existScreenshots
TODO
Who to Notify