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

[Breaking] Separate patreon integration from /info endpoint #260

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ndepaola
Copy link
Collaborator

Description

  • Motivation for this PR is the /info page drives the title on the front page, meaning this can be slow to render according to the configured site name if the Patreon API call/s take a lil while
  • This PR is an API breaking change - i.e. that the frontend and backend must be deployed together

Checklist

  • I have installed pre-commit and installed the hooks with pre-commit install before creating any commits.
  • I have updated any related tests for code I modified or added new tests where appropriate.
  • I have manually tested my changes as follows:
    • None yet, will do this later
  • I have updated any relevant documentation or created new documentation where appropriate.

@ndepaola ndepaola self-assigned this Oct 24, 2024
@ndepaola ndepaola force-pushed the separate-patreon-integration-from-/info-endpoint branch from cb68aeb to 25e948e Compare October 24, 2024 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Status: In Progress
Development

Successfully merging this pull request may close these issues.

1 participant