include title in NAME_FIELDS

Also adds support for ovveriding the filter list in `Diffable.sync`.
This is needed for milestones to work correctly. However, it should be tested well to ensure it does not break anything else.

Author: Gramatus

lib/mergeDeep.js

@@ -1,7 +1,7 @@
 const mergeBy = require('./mergeArrayBy')
 const DeploymentConfig = require('./deploymentConfig')
 
-const NAME_FIELDS = ['name', 'username', 'actor_id', 'login', 'type', 'key_prefix']
+const NAME_FIELDS = ['name', 'username', 'actor_id', 'login', 'type', 'key_prefix', 'title']
 const NAME_USERNAME_PROPERTY = item => NAME_FIELDS.find(prop => Object.prototype.hasOwnProperty.call(item, prop))
 const GET_NAME_USERNAME_PROPERTY = item => { if (NAME_USERNAME_PROPERTY(item)) return item[NAME_USERNAME_PROPERTY(item)] }
 

lib/plugins/diffable.js

@@ -24,13 +24,14 @@ const MergeDeep = require('../mergeDeep')
 const NopCommand = require('../nopcommand')
 const ignorableFields = ['id', 'node_id', 'default', 'url']
 module.exports = class Diffable extends ErrorStash {
-  constructor (nop, github, repo, entries, log, errors) {
+  constructor (nop, github, repo, entries, log, errors, filterNameFields) {
     super(errors)
     this.github = github
     this.repo = repo
     this.entries = entries
     this.log = log
     this.nop = nop
+    this.filterNameFields = filterNameFields ?? MergeDeep.NAME_FIELDS
   }
 
   filterEntries () {
@@ -102,7 +103,7 @@ module.exports = class Diffable extends ErrorStash {
           }
         }
         // Delete any diffable that now only has name and no other attributes
-        filteredEntries = filteredEntries.filter(entry => Object.keys(entry).filter(key => !MergeDeep.NAME_FIELDS.includes(key)).length !== 0)
+        filteredEntries = filteredEntries.filter(entry => Object.keys(entry).filter(key => !this.filterNameFields.includes(key)).length !== 0)
 
         const changes = []
 

lib/plugins/milestones.js

@@ -1,8 +1,9 @@
+const MergeDeep = require('../mergeDeep')
 const Diffable = require('./diffable')
 
 module.exports = class Milestones extends Diffable {
-  constructor (...args) {
-    super(...args)
+  constructor (nop, github, repo, entries, log, errors) {
+    super(nop, github, repo, entries, log, errors, MergeDeep.NAME_FIELDS.filter(i => i !== 'title'))
 
     if (this.entries) {
       this.entries.forEach(milestone => {