Open
Conversation
Add button to author screen to start the graphing.
Contributor
|
Thank you for this <3 |
Author
|
Any chance of getting this merged? <3 @rodel77 |
HexedHero
approved these changes
Jul 29, 2021
Contributor
HexedHero
left a comment
HexedHero
left a comment
There was a problem hiding this comment.
Been using this locally and had no problems.
Add queue to fix timeouts
commit 893a589 Author: rodel <rodelbusiness@hotmail.com> Date: Wed Feb 16 11:25:36 2022 -0600 Added page count in author page commit 78d3466 Merge: 7aedbda b5d32ad Author: rodel77 <rodelbusiness@hotmail.com> Date: Wed Feb 16 11:15:41 2022 -0600 Merge pull request rodel77#36 from HexedHero/prevent-too-many-requests Do one connection at a time to prevent too many requests (429) commit 7aedbda Merge: 154a0f0 1f17b74 Author: rodel77 <rodelbusiness@hotmail.com> Date: Wed Feb 16 11:10:32 2022 -0600 Merge pull request rodel77#37 from HexedHero/update-exchange-rates Update exchange rate API and default currencies commit 154a0f0 Merge: 9ea778e 79797e7 Author: rodel77 <rodelbusiness@hotmail.com> Date: Wed Feb 16 11:08:08 2022 -0600 Merge pull request rodel77#38 from HexedHero/update-high-charts Update HighCharts to v9.3.2 commit 79797e7 Author: HexedHero <6012891+HexedHero@users.noreply.github.com> Date: Sun Jan 2 02:06:10 2022 +0000 Update HighCharts to v9.3.2 commit 1f17b74 Author: HexedHero <6012891+HexedHero@users.noreply.github.com> Date: Sun Jan 2 00:06:31 2022 +0000 Use the same currencies as Spigot accepts Also update the fallback defaults to newer data from 01/01/2022 commit a1ed6b3 Author: HexedHero <6012891+HexedHero@users.noreply.github.com> Date: Sat Jan 1 23:18:37 2022 +0000 Use new API The old one went to paid only commit b5d32ad Author: HexedHero <6012891+HexedHero@users.noreply.github.com> Date: Sat Dec 18 05:54:47 2021 +0000 Change prefix commit 83a52e7 Author: HexedHero <6012891+HexedHero@users.noreply.github.com> Date: Sat Dec 18 05:51:21 2021 +0000 Do one connection at a time to prevent too many requests (429) Doing this has improved the loading times from 15-30+ mins if it didn't get stuck due to all the 429s to 25-45 seconds without a single 429 most of the time
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add button to author screen to start the graphing.
When trying to view one of my plugin pages, the page loads slowly due to the the many requests sent by the extension to load buyer information. To fix this I added a button to start the graphing requests 🙂.