Skip to content

fix: can index one to many relationships based on "indexBy"#815

Merged
nikophil merged 1 commit intozenstruck:2.3.xfrom
nikophil:fix/index-one-to-many
Feb 14, 2025
Merged

fix: can index one to many relationships based on "indexBy"#815
nikophil merged 1 commit intozenstruck:2.3.xfrom
nikophil:fix/index-one-to-many

Conversation

@nikophil
Copy link
Member

@nikophil nikophil commented Feb 13, 2025

fixes #812

ping @tandev

needs to be merged in 2.x afterwards

@nikophil nikophil force-pushed the fix/index-one-to-many branch from e25c014 to 6d54513 Compare February 13, 2025 19:29
@nikophil nikophil force-pushed the fix/index-one-to-many branch from 6d54513 to e39bee4 Compare February 13, 2025 19:33
@nikophil nikophil requested a review from kbond February 13, 2025 19:35
Copy link
Member

@kbond kbond left a comment

Choose a reason for hiding this comment

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

TIL about indexBy 😅

@nikophil nikophil merged commit ad8d72c into zenstruck:2.3.x Feb 14, 2025
66 checks passed
@nikophil nikophil deleted the fix/index-one-to-many branch February 14, 2025 06:06
nikophil added a commit that referenced this pull request Feb 14, 2025
* 2.3.x:
  changelog: update [skip ci]
  bot: fix cs [skip ci]
  fix: can index one to many relationships based on "indexBy" (#815)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants