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 button to author screen #35

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

AshleyThew
Copy link

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 🙂.

  • Dablakbandit

Add button to author screen to start the graphing.
@HexedHero
Copy link
Contributor

Thank you for this <3

@AshleyThew
Copy link
Author

Any chance of getting this merged? <3 @rodel77

Copy link
Contributor

@HexedHero HexedHero left a 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants