• 
      

    Fix a missing default and missing recorded migrations with oauth2_provider.

    Review Request #12293 — Created May 22, 2022 and submitted

    Information

    Review Board
    release-5.0.x

    Reviewers

    This fixes up some issues with the oauth2_provider upgrade.

    First, we now set an explicit default value for Grant.claims, which is
    needed on everything but MySQL and was missing in my previous change.

    We also now manage the list of recorded migrations for oauth2_provider.
    Post-upgrade, we do another clear of the list of recorded migrations and
    then we populate it with the full list that ships with oauth2_provider
    1.6.3. This avoids potential dependency issues with migrations, and it
    avoids the Django system checks complaining about missing migrations
    that still need to be applied.

    Tested a new round of upgrades. Verified that Django's system checks
    no longer saw missing migrations.

    Summary ID
    Fix a missing default and missing recorded migrations with oauth2_provider.
    This fixes up some issues with the oauth2_provider upgrade. First, we now set an explicit default value for `Grant.claims`, which is needed on everything but MySQL and was missing in my previous change. We also now manage the list of recorded migrations for oauth2_provider. Post-upgrade, we do another clear of the list of recorded migrations and then we populate it with the full list that ships with oauth2_provider 1.6.3. This avoids potential dependency issues with migrations, and it avoids the Django system checks complaining about missing migrations that still need to be applied.
    9ba355d5c83fee4dacb85e911d940a3fce895459
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-5.0.x (2019fba)