Skip to content

Conversation

jvasseur
Copy link
Contributor

@jvasseur jvasseur commented Oct 2, 2025

Problem

On a successful update, the getManyReference data update was only copying the data and total from the original response, losing other properties like meta and pageInfo.

Solution

Copy the full response like it is done for the getList response.

How To Test

Have a page that both uses a ReferenceManyField components and updates the same resource with useUpdate

Additional Checks

  • The PR targets master for a bugfix or a documentation fix, or next for a feature
  • The PR includes unit tests (if not possible, describe why)
  • The PR includes one or several stories (if not possible, describe why)
  • The documentation is up to date

…e result

On a successful update, the getManyReference data update was only
copying the data and total from the original response, losing other
properties.

This fixes it by copying the full response like it is done for the
getList response.
Copy link
Collaborator

@djhi djhi left a comment

Choose a reason for hiding this comment

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

Thanks!

@djhi djhi added this to the 5.11.5 milestone Oct 2, 2025
@fzaninotto fzaninotto merged commit 31b2780 into marmelab:master Oct 3, 2025
14 checks passed
@jvasseur jvasseur deleted the fix-update-overriding-meta-property branch October 3, 2025 07:33
@djhi djhi modified the milestones: 5.11.5, 5.12.0 Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants