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.

Sign up for GitHub

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

Jump to bottom

Include migration for SignupCode.max_uses #375

Merged

uhurusurfa merged 1 commit into master from include-migration

Sep 14, 2023

Collaborator

blag commented Sep 12, 2023

Fixes #374.

Changes proposed in this PR:

Include migration for changing default on SignupCode.max_uses to 1 (h/t @taialex1)

Tips for an ideal PR

You have read our Code of Conduct: https://github.com/pinax/.github/blob/master/CODE_OF_CONDUCT.md
You have read our Contributing info: https://github.com/pinax/.github/blob/master/CONTRIBUTING.md
The PR is atomic
Unit tests have been updated/added, if needed
App version number has been updated in setup.py (Pinax uses SemVer)
Change Log has been updated
You have added your name to the AUTHORS file


          Include migration for SignupCode.max_uses

0d6e77e

This was referenced Sep 12, 2023

Migration missing on 3.3.1 #374

Closed

Fix deepsource complaints #376

Merged

uhurusurfa approved these changes

View reviewed changes

Collaborator

uhurusurfa left a comment

👍

uhurusurfa merged commit 8c7fe02 into master

9 checks passed

blag deleted the include-migration branch

September 15, 2023 22:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet