Both is_valid and api_required are fairly dense with if-conditionals whose logic it can be difficult to parse without digging deep into it. This could benefit from refactoring.
In addition, there is no error message for when a user does not have the correct permissions to perform an action. For additional clarity, we probably want to add this.
Both
is_valid
andapi_required
are fairly dense with if-conditionals whose logic it can be difficult to parse without digging deep into it. This could benefit from refactoring.In addition, there is no error message for when a user does not have the correct permissions to perform an action. For additional clarity, we probably want to add this.