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.

Sign up for GitHub

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

Jump to bottom

V1.3.6 API Communication Improvements #131

Merged
merged 7 commits into from
Dec 27, 2020
Merged

V1.3.6 API Communication Improvements #131

merged 7 commits into from
Dec 27, 2020

Conversation

jamland
Copy link
Owner

@jamland jamland commented Dec 27, 2020 •
edited
Loading

DETAILS:
This PR mostly about improving API communication reducing the number of API requests.

CHANGES:

  • Update the app to use 4 Freesound.org API keys. This is a very dump way to scale resources.
    Since Freesound.org has limit of 2k requests daily.
  • Change default page size for requests to 30 items to reduce number of requests
  • Cache existing responses to reduce numbers of requests to freesound API
  • Attempt to set up Google Analytics 4. Need to research it more

@jamland jamland self-assigned this Dec 27, 2020
@jamland jamland merged commit d13e374 into main Dec 27, 2020
@jamland jamland deleted the v1.3.6-Analytics branch December 27, 2020 12:47
@jamland jamland changed the title V1.3.6 analytics V1.3.6 API Communication Improvements Dec 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jamland jamland

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jamland