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

[Doc] Fix useRegisterMutationMiddleware documentation #10360

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

djhi
Copy link
Collaborator

@djhi djhi commented Nov 18, 2024

Problem

Code examples in React-admin - useRegisterMutationMiddleware are erroneous because they do not return the next mutation result, although this is necessary for most main mutations handled by react admin (either create or update).

Solution

Fix the documentation.

Additional Checks

  • The PR targets master for a bugfix, 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

Also, please make sure to read the contributing guidelines.

@fzaninotto fzaninotto merged commit a17709d into master Nov 18, 2024
15 checks passed
@fzaninotto fzaninotto deleted the fix-use-register-mutation-middleware-doc branch November 18, 2024 20:34
@fzaninotto fzaninotto added this to the 5.3.5 milestone Nov 18, 2024
@djhi djhi modified the milestones: 5.3.5, 5.4.0 Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants