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

Implement Patreon API #684

Open
marksweb opened this issue Sep 15, 2021 · 7 comments
Open

Implement Patreon API #684

marksweb opened this issue Sep 15, 2021 · 7 comments

Comments

@marksweb
Copy link
Collaborator

Currently the patreon info is hardcoded and therefore a maintenance nightmare, or already out of date.

They have a python API though...

https://docs.patreon.com/#introduction

It's got to be possible to get the data we currently have hardcoded out of that API.

@amakarudze
Copy link
Contributor

I would appreciate this since I have to update manually every now and then. There other open issues though that I will close and leave this one open.

@marksweb
Copy link
Collaborator Author

@amakarudze I've had a little look at the API & in order to really get anything useful out of it, I think I'd need an access token for the django girls account.

I setup a client which is where the API keys are made available; https://www.patreon.com/portal/registration/register-clients

If a client doesn't yet exist, you just need to enter a few minor details and the website URL.

@amakarudze
Copy link
Contributor

@marksweb I have created a test-client for you use which I will delete later. Will share with you credentials privately.

@marksweb
Copy link
Collaborator Author

@amakarudze perfect, thanks 👍

@amakarudze
Copy link
Contributor

Hey @marksweb, any progress made on this issue?

@marksweb
Copy link
Collaborator Author

@amakarudze I did make a start on something for this but when I discovered we had no cache service with pythonanywhere I wasn't sure on the best way to implement.

Usually I'd cache api responses so requests weren't made on every page load.

If traffic is fairly low, that might be ok, it might even be ok to implement a local memory cache if the server has the capacity to spare.

@ramonsaraiva
Copy link
Member

I'd say that locmem can for sure be a temporary solution for this - should just be aware that each worker would have its own version of the cache - but sounds reasonable for this use case.

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

No branches or pull requests

3 participants