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 Section to README for Extensions #64

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

Add Section to README for Extensions #64

wants to merge 1 commit into from

Conversation

bam4564
Copy link

@bam4564 bam4564 commented Jan 12, 2022

Hi there! I'm the author of pycoingecko_extra which is an extension of this base api client that adds a few new features like

  • Abstracting away complexities associated with server side rate limiting when sending many api requests.
  • Page range queries.
  • Log messages.

I also have a few more future ideas of how to improve my extension of the client. My client extends the base class of this client and operates the exact same way, and the additional functionality is opt-in. Check out my README for more info.

I see my project as having a symbiotic relationship with the base client, as it merely extends instead of replacing the base api client. I am in no way am trying to replace this project, just build upon it :)

I was wondering if you would be willing to add an Extensions section to the README that links to my extension, so that users can discover it if they are interested in its feature set. This PR contains the relevant changes.

I'd also be interested to hear if you would be willing to allow me to integrate this advanced feature set into the base client itself via a future pull request. My design is a little more opinionated then the base client which is minimal, so I'd also be fine keeping the two as separate projects.

@bam4564
Copy link
Author

bam4564 commented Jan 12, 2022

@man-c @pau1a
Tagging all listed contributors for visibility.

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.

None yet

1 participant