All new code requires tests to ensure against regressions.
However, if your PR contains zero code changes, feel free to select the checkmark below to indicate so.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
It seems that we previously had a bug within the code that could stop a user from being able to publish a new package.
Essentially, we attempted to handle a specific error returned outside of the generic error handler. Meaning that when there was no error, we wouldn't encounter the specific error handling we were looking for, and would always fail the request.
This PR moves the specific error handler within the overall original error handler, so that it can function as intended.
Requirements
Filling out the template is required.
All new code requires tests to ensure against regressions.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
It seems that we previously had a bug within the code that could stop a user from being able to publish a new package.
Essentially, we attempted to handle a specific error returned outside of the generic error handler. Meaning that when there was no error, we wouldn't encounter the specific error handling we were looking for, and would always fail the request.
This PR moves the specific error handler within the overall original error handler, so that it can function as intended.
This was reported on GitHub.