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

Add wallet monitoring #55

Conversation

dylanschultzie
Copy link
Collaborator

This a rework of PFC's PR #41 to pull out the wallet monitoring functionality and clean it up a bit.

@dylanschultzie
Copy link
Collaborator Author

dylanschultzie commented Feb 24, 2023

This has only been tested with a single wallet so far, will test 2 wallets shortly.

Note that the error/OK message is changed since this screenshot.
Screenshot 2023-02-24 at 8 57 53 AM

@dylanschultzie dylanschultzie changed the title feat: Add wallet monitoring Add wallet monitoring Feb 24, 2023
@dylanschultzie dylanschultzie changed the base branch from main to develop March 9, 2023 17:24
@dylanschultzie
Copy link
Collaborator Author

Bah, switching to develop messed up the history.

@dylanschultzie
Copy link
Collaborator Author

Closing for now as it's too noisy

@joshlopes
Copy link

@dylanschultzie why is it too noisy ? I have actually recreated this (if you know me, you know using google apps scripts lol) -- but wondering if this would be good for some wallets.

(Personally I won't be moving from google, since I take snapshots of the wallets 👀 but this could be useful still for some other wallets and other people)

@dylanschultzie
Copy link
Collaborator Author

@joshlopes so, it works, but the problem is the loop in which it does the checks doesn't persist very long. If you run out of funds, it'll notify you every... 10 minutes?

@joshlopes
Copy link

@dylanschultzie That sounds a quick in-memory / temp file fix? for how long you want it to alert or in betweens? Sorry for not being too much help... I hate GO - doesn't make sense to me the structure so messy 🙈

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.

2 participants