Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merging props documentation #379

Open
DarkVen0m opened this issue Jan 5, 2025 · 0 comments
Open

Merging props documentation #379

DarkVen0m opened this issue Jan 5, 2025 · 0 comments

Comments

@DarkVen0m
Copy link

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.
https://github.com/inertiajs/inertia/blob/19b6ab40eca82d900c2f0b94e62fb4f1847c073d/packages/core/src/response.ts#L240
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.

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

No branches or pull requests

1 participant