-
Notifications
You must be signed in to change notification settings - Fork 10
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
Auto-Updater? #29
Comments
So, the way that it works is as follows: |
That's totally fair- and honestly I value the accuracy over the speed as I said earlier. Much prefer being able to get reliable information consistently rather than having to remember the exact number. Might as well just memorize the values then, and that would make the whole thing redundant! Definitely, appreciate the time and effort you have put in, and while I hope the auto-updater does come soon I can be patient! |
Fyi, working on it. |
That is awesome! |
@FerrocCaldrson For now there is not way of easy controlling autoupdating, it's always on and updates roughly every 30 seconds or so. If you want to change it, set new update value on line 397. Adding UI should be easy now, but will probably take a few more days as I do not have a lot of spare time when I can still think clearly. :) Let me know if you like it. |
I mean I personally don't mind the look at all. And it works great! I've tested it out with a couple of characters in my campaign and it does do the job. I noticed the updater has the option of going less than 30, but I assume (and testing it out I think I'm pretty sure) the fastest it can update is 30. Nothing wrong with that at all, but if someone does end up going in and "fixing" it to look all pretty, may wanna have them cap the lowest option at thirty. But honestly great job, this will make game nights a lot easier! Thanks so much! |
Actually no, it's just the way it works right now that might give this impression. After the function that retrieves data and draws it on screen a Glad you like it! :) |
@FerrocCaldrson, could I ask you to test newest version? It's the same link as above. I added config persistency, so now update period you set once should work even after restarts/refreshes. |
Just tested it, timer configuration persists even after closing and opening the page in a new tab. Very cool addition, thanks! |
@FerrocCaldrson, there has been a major update bone by @TeaWithLucas, I think you want to update to their version. https://github.com/TeaWithLucas/DNDBeyond-DM-Screen |
Thanks for the heads up! |
Hey, so been chatting with you on the other guy's GitHub, and honestly I'm liking your original version better. No errors in information displayed at all. As per our discussion, an chance of adding an auto-updater, so that one does not need to continually refresh the page every few minutes?
Also the loading times are a little long, but this is sorta secondary. Just figured if you have time and it is something you can tweak, might as well mention it.
The text was updated successfully, but these errors were encountered: