search

cambiatus / backend

Cambiatus GraphQL API
GNU Affero General Public License v3.0
21 stars 18 forks source link

Give errors to Oban as 2 tuples #305

Closed seth-schroeder closed 1 year ago

seth-schroeder commented 1 year ago

Please note

This is a subset of cambiatus/backend#306

What issue does this PR close

Closes cambiatus/backend#304

Changes Proposed ( a list of new changes introduced by this PR)

Return 2 tuple errors to Oban. 4 tuple errors are generated when Multi.new is upstream of Repo.transaction.

How to test ( a list of instructions on how to test this PR)

Existing unit test coverage. They passed for me locally, after running mix format and mix credo.

Please note

I am submitting two versions of this change. This PR addresses the error handling. The next PR will have that, and it will include the credo check.