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

Plugin documentation #155

Merged
merged 5 commits into from
Mar 21, 2023
Merged

Conversation

abkfenris
Copy link
Member

I've introduced the plugin system with descriptions and examples for the 4 main kinds of plugins (app/dataset routers, dataset providers, and hook spec). I've also tried to explain the common functionality and enough of the underlying system to get folks started building their own.

@kwilcox @mpiannucci I'd love your feedback as I'm pretty sure you are both using plugins, and I'd love to hear from anyone else using plugins to about these docs and if there is anything that would help improve them.

I've introduced the plugin system with descriptions and examples for the 4 main kinds of plugins (app/dataset routers, dataset providers, and hook spec). I've also tried to explain the common functionality and enough of the underlying system to get folks started building their own.
@abkfenris
Copy link
Member Author

abkfenris commented Mar 6, 2023

Fixed: Ok, not sure why RTD is not rendering bullets:

Local vs RTD
image image

@jr3cermak
Copy link

Great work!

One update to the docs, include self for the function (L315; docs/source/plugins.rst):

def get_dataset(self, dataset_id: str):

instead of:

def get_dataset(dataset_id: str):

I kept chasing my tail for a while wondering why dataset_id wasn't getting anything but the self object when I was attempting to work with the plugins.

@abkfenris
Copy link
Member Author

Whoops, thanks for catching that @jr3cermak!

@abkfenris
Copy link
Member Author

Ok, bullets are working on plugin and contributing pages. RTD has a really sticky cache for PR previews.

image

@abkfenris abkfenris added the documentation Improvements or additions to documentation label Mar 9, 2023
@abkfenris abkfenris mentioned this pull request Mar 9, 2023
9 tasks
@abkfenris abkfenris merged commit 2667b64 into xpublish-community:main Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants