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.

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

fix BD-3236 BD-3374, BD-3404 #345

Merged
merged 2 commits into from
Mar 8, 2022

Conversation

edrotthoff
Copy link
Contributor

@edrotthoff edrotthoff commented Mar 4, 2022

fix bug in avail_check to return is registered when a domain is present and expired, fix bug in get fio balance when no currency found for this account, return available 0

also resolve issue existing and expired fio handles get is_registered 0 (mod code to return is_registered true when fio handle exists and is expired)

NOTE -- this needs a complete regression test before it should be considered for hot fix.
NOTE -- this needs a complete regression test before it should be considered for hot fix.
NOTE -- this needs a complete regression test before it should be considered for hot fix.

fix bug in avail_check to return is registered when a domain is present and expired, fix bug in get fio balance when no currency found for this account, return available 0
@edrotthoff edrotthoff marked this pull request as draft March 4, 2022 02:23
return is_registered true for expired addresses
@edrotthoff edrotthoff changed the title fix BD-3236 BD-3374 fix BD-3236 BD-3374, return is_registered true for existing and expired fio addresses/handles Mar 4, 2022
@edrotthoff edrotthoff changed the title fix BD-3236 BD-3374, return is_registered true for existing and expired fio addresses/handles fix BD-3236 BD-3374, BD-3404 Mar 6, 2022
@ericbutz ericbutz marked this pull request as ready for review March 8, 2022 13:42
@ericbutz ericbutz merged commit 817447f into release/3.3.x-dev Mar 8, 2022
@ericbutz ericbutz deleted the feature/BD-3236-3374-03032022 branch March 8, 2022 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants