-
Notifications
You must be signed in to change notification settings - Fork 7
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 button to author screen #35
Open
AshleyThew
wants to merge
3
commits into
rodel77:master
Choose a base branch
from
AshleyThew:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
Add button to author screen to start the graphing.
Thank you for this <3 |
Any chance of getting this merged? <3 @rodel77 |
HexedHero
approved these changes
Jul 29, 2021
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Been using this locally and had no problems.
Add queue to fix timeouts
commit 893a589 Author: rodel <[email protected]> Date: Wed Feb 16 11:25:36 2022 -0600 Added page count in author page commit 78d3466 Merge: 7aedbda b5d32ad Author: rodel77 <[email protected]> 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 <[email protected]> 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 <[email protected]> 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 <[email protected]> Date: Sun Jan 2 02:06:10 2022 +0000 Update HighCharts to v9.3.2 commit 1f17b74 Author: HexedHero <[email protected]> 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 <[email protected]> Date: Sat Jan 1 23:18:37 2022 +0000 Use new API The old one went to paid only commit b5d32ad Author: HexedHero <[email protected]> Date: Sat Dec 18 05:54:47 2021 +0000 Change prefix commit 83a52e7 Author: HexedHero <[email protected]> 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 🙂.