Django3: remove old non-JSONFields and kept the new ones only

```
In [3]: Build.objects.filter(_config__build__image='readthedocs/build:latest')
Out[3]: <BuildQuerySet [<Build: Build qtile for admin (2)>, <Build: Build qtile for admin (1)>]>
```

Author: humitos

readthedocs/builds/migrations/0039_remove_old_jsonfields.py

@@ -0,0 +1,22 @@
+# Generated by Django 3.2.11 on 2022-01-31 12:12
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('builds', '0038_add_new_jsonfields'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='build',
+            name='_config',
+        ),
+        migrations.RenameField(
+            model_name='build',
+            old_name='_config_json',
+            new_name='_config',
+        ),
+    ]

readthedocs/builds/models.py

@@ -20,7 +20,6 @@
     ModificationDateTimeField,
 )
 from django_extensions.db.models import TimeStampedModel
-from jsonfield import JSONField
 from polymorphic.models import PolymorphicModel
 
 import readthedocs.builds.automation_actions as actions
@@ -647,8 +646,7 @@ class Build(models.Model):
         null=True,
         blank=True,
     )
-    _config = JSONField(_('Configuration used in the build'), default=dict)
-    _config_json = models.JSONField(_('Configuration used in the build'), default=dict)
+    _config = models.JSONField(_('Configuration used in the build'), default=dict)
 
     length = models.IntegerField(_('Build Length'), null=True, blank=True)
 

readthedocs/integrations/migrations/0009_remove_old_jsonfields.py

@@ -0,0 +1,40 @@
+# Generated by Django 3.2.11 on 2022-01-31 12:12
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('integrations', '0008_add_new_jsonfields'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='httpexchange',
+            name='request_headers',
+        ),
+        migrations.RemoveField(
+            model_name='httpexchange',
+            name='response_headers',
+        ),
+        migrations.RemoveField(
+            model_name='integration',
+            name='provider_data',
+        ),
+        migrations.RenameField(
+            model_name='httpexchange',
+            old_name='request_headers_json',
+            new_name='request_headers',
+        ),
+        migrations.RenameField(
+            model_name='httpexchange',
+            old_name='response_headers_json',
+            new_name='response_headers',
+        ),
+        migrations.RenameField(
+            model_name='integration',
+            old_name='provider_data_json',
+            new_name='provider_data',
+        ),
+    ]

readthedocs/integrations/models.py

@@ -12,7 +12,6 @@
 from django.db import models, transaction
 from django.utils.safestring import mark_safe
 from django.utils.translation import gettext_lazy as _
-from jsonfield import JSONField
 from pygments import highlight
 from pygments.formatters import HtmlFormatter
 from pygments.lexers import JsonLexer
@@ -144,17 +143,15 @@ class HttpExchange(models.Model):
 
     date = models.DateTimeField(_('Date'), auto_now_add=True)
 
-    request_headers = JSONField(_('Request headers'))
-    request_headers_json = models.JSONField(
+    request_headers = models.JSONField(
         _('Request headers'),
         # Delete after deploy
         null=True,
         default=None,
     )
     request_body = models.TextField(_('Request body'))
 
-    response_headers = JSONField(_('Request headers'))
-    response_headers_json = models.JSONField(
+    response_headers = models.JSONField(
         _('Request headers'),
         # Delete after deploy
         null=True,
@@ -289,8 +286,7 @@ class Integration(models.Model):
         max_length=32,
         choices=INTEGRATIONS,
     )
-    provider_data = JSONField(_('Provider data'), default=dict)
-    provider_data_json = models.JSONField(_('Provider data'), default=dict)
+    provider_data = models.JSONField(_('Provider data'), default=dict)
     exchanges = GenericRelation(
         'HttpExchange',
         related_query_name='integrations',

requirements/pip.txt

@@ -21,6 +21,10 @@ drf-flex-fields==0.9.7
 drf-extensions==0.7.1
 
 django-vanilla-views==3.0.0
+
+# This module is only used on migrations. We are now using Django's official
+# JSONField. We should probably squash these migrations and remove this
+# dependency as well.
 jsonfield==3.1.0
 
 requests==2.27.1