Setting version in the patch object and the header object

[YashSinghMk]

After upgrading to from 4.29.1 to 4.29.5 and specifically running a deduplication. We were getting a build error where ts was looking for the version prop that was not provided. After adding version to patch objects the build is running successfully. Are we now supposed to be setting version in the patch object and the header object? Or can we drop the header object here?

contentfulClient.entry.patch(
 {
    entryId: entry.sys.id,
    version: entry.sys.version // Do we need this...
  },
  operations,
  {
     'X-Contentful-Version': entry.sys.version // ... AND this?
   }
)

We couldn't find any updates in the docs or release notes.

[whitelisab]

Hi Yash Singh (@YashSinghMk)! Thanks for bringing this to our attention. We've made a change in contentful-management v11.52.1 that will make the version parameter optional. We also bumped contentful-management in the @contentful/app-sdk, so try v4.29.6 of that package to see this change.

The code in this contentful-management package expects that version is passed in as a parameter for the entry patch method. We plan to release a new major version of this package where this is required and update our documentation accordingly. We would encourage you to start moving towards passing the version in the patch parameter object instead of in the header. Then in the new major version of this package, version will be a required on the parmeters object.

Let me know if you need any further information, and thanks again for reaching out!