Skip to content

Conversation

@r1tsuu
Copy link
Member

@r1tsuu r1tsuu commented Sep 20, 2024

Fixes #7695

Previosuly, trying to append a new item to an array that contains another array with localized items and enabled versions led to a unique _locale and _parent_id error

{
  name: 'nestedArrayLocalized',
  type: 'array',
  fields: [
    {
      type: 'array',
      name: 'array',
      fields: [
        {
          name: 'text',
          type: 'text',
          localized: true,
        },
      ],
    },
  ],
}

@github-actions github-actions bot added the v3 label Sep 20, 2024
@r1tsuu r1tsuu requested a review from DanRibbens September 20, 2024 14:26
@DanRibbens DanRibbens merged commit 2596b85 into beta Sep 20, 2024
@DanRibbens DanRibbens deleted the fix/pg-nested-array-localized branch September 20, 2024 15:16
DanRibbens pushed a commit that referenced this pull request Sep 20, 2024
Port of #8331 to 2.0

Previosuly, trying to append a new item to an array that contains
another array with localized items and enabled versions led to a unique
`_locale` and `_parent_id` error
```ts
{
  name: 'nestedArrayLocalized',
  type: 'array',
  fields: [
    {
      type: 'array',
      name: 'array',
      fields: [
        {
          name: 'text',
          type: 'text',
          localized: true,
        },
      ],
    },
  ],
}
```
@github-actions
Copy link
Contributor

🚀 This is included in version v3.0.0-beta.108

DanRibbens added a commit that referenced this pull request Oct 8, 2024
This fixes many various issues that are already fixed in 3.0. Updates
Drizzle to match beta to fix some issues
#4673
#6845
#6266, prev Drizzle update
PR #7460

Ported PRs:
- #6158
- #7900
- #7962 (does include
duplication fixes for blocks / arrays with specific for 2.0 method)
- #8355
- #8456
- #8331 (not in the commits
list, as it was a clean merge)
- #8369
- #7749
- #8539

---------

Co-authored-by: Dan Ribbens <dan.ribbens@gmail.com>
Co-authored-by: James Mikrut <james@payloadcms.com>
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.

3 participants