Skip to content

Process multiple update for the same new entity #4890

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

Conversation

demyanm
Copy link

@demyanm demyanm commented Mar 12, 2025

Fixes #3429

Copy link

codesandbox bot commented Mar 12, 2025

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

Copy link

codesandbox-ci bot commented Mar 12, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit e0ef8c5:

Sandbox Source
@examples-query-react/basic Configuration
@examples-query-react/advanced Configuration
@examples-action-listener/counter Configuration
rtk-esm-cra Configuration

Copy link

netlify bot commented Mar 12, 2025

Deploy Preview for redux-starter-kit-docs ready!

Name Link
🔨 Latest commit e0ef8c5
🔍 Latest deploy log https://app.netlify.com/sites/redux-starter-kit-docs/deploys/67d1ab8176f13c00089ec738
😎 Deploy Preview https://deploy-preview-4890--redux-starter-kit-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -0,0 +1,53 @@
import { createAction, createEntityAdapter, type EntityId } from '@reduxjs/toolkit'

describe('createEntityAdapter', () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

tests for createEntityAdapter are in packages/toolkit/src/entities/tests

@demyanm demyanm requested a review from EskiMojo14 March 13, 2025 09:35
Copy link
Collaborator

@EskiMojo14 EskiMojo14 left a comment

Choose a reason for hiding this comment

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

makes sense, thanks!

@EskiMojo14 EskiMojo14 merged commit d6f72d7 into reduxjs:master Mar 13, 2025
47 checks passed
@demyanm demyanm deleted the bugfix/3429-process-multiple-update-for-the-same-new-entity branch March 13, 2025 11:30
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.

upsertMany with multiple updates with the same id throws away updates
2 participants