extendField should extend field extensions

[stefanprobst]

Currently, extendField overwrites an existing extensions field property. I think it would be better if both extensions are merged?

const { ObjectTypeComposer, SchemaComposer } = require('graphql-compose');

const sc = new SchemaComposer();

const tc = ObjectTypeComposer.create(
  {
    name: 'Test',
    fields: {
      test: {
        type: 'String',
        extensions: { test: true },
      },
    },
  },
  sc
);

tc.extendField('test', {
  resolve: source => source.test,
  extensions: { added: true },
});

console.log(tc.getFieldExtensions('test'));
// { added: true }
// Should be { test: true, added: true }

[nodkz]

Agreed with you. 👍
Can you sync with @freiksenet with this change?

And he also agreed can you make Pull Request?

[nodkz]

I think that he is sleeping right now. So you may make Pull Request and we await his approving.

[freiksenet]

Yeah, I agree this is a better behavior. This is just an oversight on my part.

[nodkz]

🎉 This issue has been resolved in version 6.1.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀