-
Notifications
You must be signed in to change notification settings - Fork 203
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhancement: Boxes Indexer Integration test #641
Enhancement: Boxes Indexer Integration test #641
Conversation
2c1076d
to
2f1083a
Compare
347042e
to
85641b8
Compare
afaac2d
to
7211d3d
Compare
7211d3d
to
9713f8f
Compare
7409af5
to
7e2a62b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Basically approvable but waiting on decision for future of tag @applications.boxes_indexer_confirmed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LEBTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comment
…nd/js-algorand-sdk into ahangsu/indexer-integration-boxes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - just a reminder for the other assert.fail
s
…nd/js-algorand-sdk into ahangsu/indexer-integration-boxes
…nd/js-algorand-sdk into ahangsu/indexer-integration-boxes
No description provided.