remove additional information about deleted fields

src/lib/action/content-type-field-purge.ts

@@ -0,0 +1,20 @@
+import { APIAction } from './action'
+import OfflineAPI from '../offline-api/index'
+
+class ContentTypeFieldPurgeAction extends APIAction {
+  private contentTypeId: string
+  private fieldId: string
+
+  constructor (contentTypeId: string, fieldId: string) {
+    super()
+    this.contentTypeId = contentTypeId
+    this.fieldId = fieldId
+  }
+
+  async applyTo (api: OfflineAPI) {
+    const contentType = await api.getContentType(this.contentTypeId)
+    contentType.fields.deleteField(this.fieldId)
+  }
+}
+
+export { ContentTypeFieldPurgeAction }

src/lib/intent/field-delete.ts

@@ -3,6 +3,7 @@ import { FieldUpdateAction } from '../action/field-update'
 import { ContentTypeSaveAction } from '../action/content-type-save'
 import { ContentTypePublishAction } from '../action/content-type-publish'
 import { EntryFieldPurgeAction } from '../action/entry-field-purge'
+import { ContentTypeFieldPurgeAction } from '../action/content-type-field-purge'
 import { PlanMessage } from '../interfaces/plan-message'
 import chalk from 'chalk'
 
@@ -41,6 +42,10 @@ export default class FieldDeleteIntent extends Intent {
       new EntryFieldPurgeAction(
         contentTypeId,
         fieldId
+      ),
+      new ContentTypeFieldPurgeAction(
+        contentTypeId,
+        fieldId
       )
     ]
   }

test/fixtures/contentful-migration-integration.js

@@ -1398,7 +1398,7 @@ nock('https://api.contentful.com:443', {"encodedQueryParams":true})
 
 
 nock('https://api.contentful.com:443', {"encodedQueryParams":true})
-  .put('/spaces/bohepdihyxin/environments/env-integration/content_types/dog', {"name":"angry dog","displayField":null,"fields":[{"id":"woofs","name":"woof woof","type":"Number","localized":false,"required":true,"validations":[],"disabled":false,"omitted":true,"deleted":true}],"description":"super angry"})
+  .put('/spaces/bohepdihyxin/environments/env-integration/content_types/dog', {"name":"angry dog","displayField":null,"fields":[],"description":"super angry"})
   .reply(200, {
   "sys": {
     "space": {

test/unit/lib/offline-api/request-batches.spec.ts

@@ -142,14 +142,6 @@ describe('Payload builder', function () {
               type: 'Number',
               name: 'kills',
               required: true
-            },
-            {
-              id: 'favoriteFood',
-              type: 'Symbol',
-              name: 'food',
-              required: true,
-              omitted: true,
-              deleted: true
             }
           ]
         }
@@ -178,13 +170,6 @@ describe('Payload builder', function () {
                 required: true
               },
               {
-                id: 'favoriteFood',
-                type: 'Symbol',
-                name: 'food',
-                required: true,
-                omitted: true,
-                deleted: true
-              }, {
                 id: 'foo',
                 type: 'Symbol',
                 name: 'A foo'

test/unit/lib/offline-api/validation/payload-validation-display-field.spec.ts

@@ -66,7 +66,7 @@ describe('payload validation (display field)', function () {
         [
           {
             type: 'InvalidAction',
-            message: 'Cannot delete field "dessert" on content type "lunch" because it is set as the display field'
+            message: 'Cannot set "dessert" as displayField on content type "lunch" because it does not exist'
           }
         ]
       ])