Skip to content

Commit

Permalink
Fixed bug where method is_accounts_refresh_complete returned the wr…
Browse files Browse the repository at this point in the history
…ong value. The method previously return true if every account did not have its' sync in progress. I removed the `not` from before the return statement so that now it returns true if all accounts have started their sync. I am relatively certain that this is what the code should do, please close this branch if not.
  • Loading branch information
MichalikSJ21 committed May 11, 2024
1 parent 655b8fa commit 83446ab
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions monarchmoney/monarchmoney.py
Original file line number Diff line number Diff line change
Expand Up @@ -686,13 +686,13 @@ async def is_accounts_refresh_complete(
if account_ids:
return all(
[
not x["hasSyncInProgress"]
x["hasSyncInProgress"]
for x in response["accounts"]
if x["id"] in account_ids
]
)
else:
return all([not x["hasSyncInProgress"] for x in response["accounts"]])
return all([x["hasSyncInProgress"] for x in response["accounts"]])

async def request_accounts_refresh_and_wait(
self,
Expand Down

0 comments on commit 83446ab

Please sign in to comment.