Add debug for multilookup bindings bug (!395) · Merge requests · elixxir / client · GitLab

Snippets Groups Projects

Merged Josh Brooks requested to merge XX-4216/MultiLookup into release 2 years ago

All threads resolved!

Simple bug fix for XX-4216. The bug was a poorly constructed error handling, which would return an error on a successful MultiLookup.

Edited 2 years ago by Josh Brooks

Activity

Josh Brooks added 1 commit 2 years ago
added 1 commit

911c48dc - Attempt to fix multilookup

Compare with previous version
Josh Brooks added 1 commit 2 years ago
added 1 commit

d95a80ff - Add more debugs

Compare with previous version
Josh Brooks added 1 commit 2 years ago
added 1 commit

fcde7708 - Add more debug prints

Compare with previous version
Josh Brooks added 2 commits 2 years ago
added 2 commits

a282548a - Clean up

e93a305c - Clean up API

Compare with previous version
Josh Brooks changed the description 2 years ago

changed the description
Richard T. Carback III approved this merge request 2 years ago

approved this merge request
Richard T. Carback III @carback1 · 2 years ago

Maintainer

Resolved 2 years ago by Richard T. Carback III

LGTM -- Why are we removing the batch lookup? I think something downstream might use it?

Last reply by Josh Brooks 2 years ago
Richard T. Carback III resolved all threads 2 years ago

resolved all threads
Richard T. Carback III mentioned in commit 0c26c935 2 years ago

mentioned in commit 0c26c935
Richard T. Carback III merged 2 years ago

merged

Please register or sign in to reply