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

Explicit default_auto_field. #326

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

Commits on Jan 25, 2024

  1. Explicit default_auto_field.

    This is configurable since Django 3.2, which is already the required
    version for oscar-api.
    
    see:
    https://docs.djangoproject.com/en/3.2/topics/db/models/#automatic-primary-key-fields
    
    Explicitly telling which field to use if good, it avoids a supious
    migration like:
    
        $ ./manage.py makemigrations
        Migrations for 'oscarapi':
          /home/mdk/src/django-oscar/.venv/lib/python3.11/site-packages/oscarapi/migrations/0002_alter_apikey_id.py
            - Alter field id on apikey
    
    notice the migration is created in the venv, in oscarapi tree, not in
    the user project, it won't work anyway.
    JulienPalard committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    d965240 View commit details
    Browse the repository at this point in the history