Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enable 0 value for currency-greater-than #810

Closed
shiqizng opened this issue Aug 10, 2023 · 0 comments · Fixed by #807
Closed

Enable 0 value for currency-greater-than #810

shiqizng opened this issue Aug 10, 2023 · 0 comments · Fixed by #807
Assignees
@shiqizng
Labels
new-feature-request Feature request that needs triage Team Lamprey

Comments

@shiqizng
Copy link
Contributor

shiqizng commented Aug 10, 2023 •
edited
Loading

Problem

When making the call client.lookupAssetBalances(asset_id).currencyGreaterThan(0).do() the request has no currency-greater-than

Solution

Until #414 is resolved, treat 0 as '0' to get around this issue.
@shiqizng shiqizng added new-feature-request Feature request that needs triage Team Lamprey labels Aug 10, 2023
@shiqizng shiqizng changed the title Remove removeFalsyOrEmpty Enable 0 value for currency-greater-than Aug 10, 2023
@shiqizng shiqizng mentioned this issue Aug 8, 2023
Merged
@shiqizng shiqizng self-assigned this Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shiqizng shiqizng

Labels
new-feature-request Feature request that needs triage Team Lamprey
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bug-fix: include currency-greater-than param for 0 value algorand/js-algorand-sdk
1 participant
@shiqizng