Merging props documentation

[DarkVen0m]

Versions:

Inertia 2.0 documentation

Describe the problem:

Documentation on merging props
https://inertiajs.com/merging-props
Does not mention that it only works with partial reloads, which requires either only/except or reset parameter.

inertia/packages/core/src/response.ts

Line 240 in 19b6ab4

if (this.requestParams.isPartial() && pageResponse.component === currentPage.get().component) {

In my opinion it should mention it, as from front-end view everything looks okay without specifying those parameters, mergeProps is present in response as well as prop itself.

Steps to reproduce:

Make any "full"-request with merge props - it won't work.

[HichemTab-tech]

Hi @DarkVen0m can you please create an issue in inertia documentation GitHub https://github.com/inertiajs/inertiajs.com? Thanks!!.

And by the way, the merge feature has other problems, it isn't working correctly with paging format , u can check the issue with proposed solutions here : #2068

[DarkVen0m]

Hi @DarkVen0m can you please create an issue in inertia documentation GitHub https://github.com/inertiajs/inertiajs.com? Thanks!!.

And by the way, the merge feature has other problems, it isn't working correctly with paging format , u can check the issue with proposed solutions here : #2068

Thanks, will do.
Yeah, I saw other issues while investigating why props merging doesn't work for me. But in my case it's pagination on my end with simple array/Collection, no nested objects. So specifying "only" parameter did the trick.
The only point was that it's not mentioned in documentation, so as last resort I had to go to inertia source code to understand why exactly it wasn't working.