search

hammem / monarchmoney

Python API for Monarch Money
MIT License
124 stars 24 forks source link

Account Refresh Bugfix #105

Closed MichalikSJ21 closed 2 months ago

MichalikSJ21 commented 2 months ago

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.

MichalikSJ21 commented 2 months ago

@hammem That makes sense, thank you for clarifying that. I'll close this branch, I think that my code does not accomplish that result. Thanks for clarifying that for me!

hammem commented 2 months ago

No worries! Thanks for the second set of eyes on the code 👀