Contributions are welcome, and they are greatly appreciated! Every little bit helps, and credit will always be given.
You can contribute in many ways:
Types of Contributions¶
Report bugs at https://github.com/dj-stripe/dj-stripe/issues.
If you are reporting a bug, please include:
- The version of python and Django you’re running
- Detailed steps to reproduce the bug.
Look through the GitHub issues for bugs. Anything tagged with “bug” is open to whoever wants to implement it.
Look through the GitHub issues for features. Anything tagged with “feature” is open to whoever wants to implement it.
dj-stripe could always use more documentation, whether as part of the official dj-stripe docs, in docstrings, or even on the web in blog posts, articles, and such.
If you are adding to dj-stripe’s documentation, you can see your changes by
tox -e docs. The documentation will be generated in the
directory, and you can open
docs/_build/html/index.html in a web browser.
The best way to send feedback is to file an issue at https://github.com/dj-stripe/dj-stripe/issues.
If you are proposing a feature:
- Explain in detail how it would work.
- Keep the scope as narrow as possible, to make it easier to implement.
- Remember that this is a volunteer-driven project, and that contributions are welcome :)
Ready to contribute? Here’s how to set up dj-stripe for local development.
Fork the dj-stripe repo on GitHub.
Clone your fork locally:
$ git clone email@example.com:your_name_here/dj-stripe.git
Set up your test database. If you’re running tests using PostgreSQL:
$ createdb djstripe
or if you want to test vs sqlite (for convenience) or MySQL, they can be selected by setting this environment variable:
$ export DJSTRIPE_TEST_DB_VENDOR=sqlite
$ export DJSTRIPE_TEST_DB_VENDOR=mysql
For postgres and mysql, the database host,port,username and password can be set with environment variables, see
Install your local copy into a virtualenv. Assuming you have
virtualenvwrapperinstalled, this is how you set up your fork for local development:
$ mkvirtualenv dj-stripe $ cd dj-stripe/ $ python setup.py develop
Create a branch for local development:
$ git checkout -b name-of-your-bugfix-or-feature
Now you can make your changes locally.
When you’re done making changes, check that your changes pass the tests. A quick test run can be done as follows:
$ DJSTRIPE_TEST_DB_VENDOR=sqlite pytest --reuse-db
You should also check that the tests pass with other python and Django versions with tox. pytest will output both command line and html coverage statistics and will warn you if your changes caused code coverage to drop.:
$ pip install tox $ tox
If your changes altered the models you may need to generate Django migrations:
$ DJSTRIPE_TEST_DB_VENDOR=sqlite ./manage.py makemigrations
Commit your changes and push your branch to GitHub:
$ git add . $ git commit -m "Your detailed description of your changes." $ git push origin name-of-your-bugfix-or-feature
Submit a pull request through the GitHub website.
Congratulations, you’re now a dj-stripe contributor! Have some <3 from us.
Preferred Django Model Field Types¶
When mapping from Stripe API field types to Django model fields, we try to follow Django best practises where practical.
The following types should be preferred for fields that map to the Stripe API (which is almost all fields in our models).
- Stripe API string fields have a default maximum length of 5,000 characters.
- In some cases a maximum length (
maxLength) is specified in the Stripe OpenAPI schema.
- We follow Django’s recommendation
and avoid using null on string fields (which means we store
""for string fields that are
nullin stripe). Note that is enforced in the sync logic in StripeModel._stripe_object_to_record.
- For long string fields (eg above 255 characters) we prefer
Therefore the default type for string fields that don’t have a maxLength specified in the Stripe OpenAPI schema should usually be:
str_field = TextField(max_length=5000, default=", blank=True, help_text="...")
Fields that have a defined set of values can be implemented using
djstripe.fields, see also the
Stripe handles all currency amounts as integer cents, we currently have a mixture of fields as integer cents and decimal (eg dollar, euro etc) values, but we are aiming to standardise on cents (see https://github.com/dj-stripe/dj-stripe/issues/955).
All new currency amount fields should use
Dates and Datetimes¶
The Stripe API uses an integer timestamp (seconds since the Unix epoch) for dates and
datetimes. We store this as a datetime field, using
Django Migration Policy¶
Migrations are considered a breaking change, so it’s not usually not acceptable to add a migration to a stable branch,
it will be a new
A workaround to this in the case that the Stripe API data isn’t compatible with out model (eg Stripe is sending
null to a non-null field)
is to implement the
_manipulate_stripe_object_hook classmethod on the model.
Avoid new migrations with non-schema changes¶
If a code change produces a migration that doesn’t alter the database schema (eg changing
help_text) then instead of
adding a new migration you can edit the most recent migration that affects the field in question.
Squash of unreleased migrations on master¶
We aim to keep the number of migration files per release to a minimum per
In the case where there are several unreleased migrations on master between releases, we squash migrations immediately before release.
So if you’re using the master branch with unreleased migrations, ensure you migrate with the squashed migration before upgrading to the next major release.
For more details see the Squash migrations section of the Release process.
Pull Request Guidelines¶
Before you submit a pull request, check that it meets these guidelines:
- The pull request should include tests.
- The pull request must not drop code coverage below the current level.
- If the pull request adds functionality, the docs should be updated. Put your new functionality into a function with a docstring.
- If the pull request makes changes to a model, include Django migrations.
- The pull request should work for Python 3.6+. Check https://travis-ci.org/dj-stripe/dj-stripe/pull_requests and make sure that the tests pass for all supported Python versions.
- Code formatting: Make sure to install
pip install black isortand run
black .; isort -yat the dj-stripe root to keep a consistent style.