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

Docs/custom providers #36

Merged
merged 14 commits into from
Oct 16, 2018
Merged

Docs/custom providers #36

merged 14 commits into from
Oct 16, 2018

Conversation

mattjegan
Copy link
Collaborator

@mattjegan mattjegan commented Oct 2, 2018

Adds initial documentation for how to implement a custom provider. I imagine that adding docstrings to BaseProvider will also help but for now I think that a basic snippet is enough. It also fixes a dependancy error in the Pipfile where the incorrect django package was being installed.

This PR depends on #26 and #27 being merged to master first.

Fixes #31

@mattjegan mattjegan added this to the 1.0.0 milestone Oct 2, 2018
@mattjegan mattjegan self-assigned this Oct 2, 2018
@mattjegan mattjegan requested a review from aljp October 2, 2018 08:55
@mattjegan mattjegan merged commit de6cd83 into master Oct 16, 2018
@mattjegan mattjegan deleted the docs/custom-providers branch October 16, 2018 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add docs for "Implementing custom providers"
2 participants