From 83446abfc70df4254f4017e75470b25b079cafeb Mon Sep 17 00:00:00 2001 From: Stevie Michalik Date: Fri, 10 May 2024 21:16:22 -0400 Subject: [PATCH] Fixed bug where method `is_accounts_refresh_complete` returned the wrong 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. --- monarchmoney/monarchmoney.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/monarchmoney/monarchmoney.py b/monarchmoney/monarchmoney.py index d5ee5d1..922a36f 100644 --- a/monarchmoney/monarchmoney.py +++ b/monarchmoney/monarchmoney.py @@ -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,