Skip to content

Conversation

@jberry92
Copy link

@jberry92 jberry92 commented Dec 5, 2025

Summary

Exposing referencingEntryId for resources on contentful management client

Description

The Extensibility API exposes referencingEntryId to enable passing to invoked functions for customers to do additional computation, however it is not exposed on this client which is used in other places to invoke the call to the management API. This PR exposes this new optional field on the management client.

Motivation and Context

Customers sometimes want to pass on the referencingEntryId for Native External References to do additional lookups or computation.

PR Checklist

  • I have read the CONTRIBUTING.md file
  • All commits follow conventional commits
  • Documentation is updated (if necessary)
  • PR doesn't contain any sensitive information
  • There are no breaking changes

@jberry92 jberry92 requested a review from a team as a code owner December 5, 2025 15:53
@elylucasctfl elylucasctfl changed the title [ARC-727] Expose referencingEntryId on resources feat: expose referencingEntryId on resources [ARC-727] Dec 8, 2025
environmentId,
resourceTypeId,
query: {
'sys.urn[in]': '123,456',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

curious if this test was suppose to include the referencingEntryId?

Copy link
Author

@jberry92 jberry92 Dec 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a test that makes some sense in that locale can be an optional parameter as well without the referencingEntryId - probably not one I would have added, I think copilot got a bit over excited and just tested all possibilities.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds good, feel free to merge when you are ready

Copy link
Contributor

@elylucasctfl elylucasctfl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left a comment about a test but besides that 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants