Skip to content

fix(pinia-orm): multiple belongsToMany relations still wrong saved#373

Merged
CodeDredd merged 1 commit intomasterfrom
pinia-orm-363
Sep 20, 2022
Merged

fix(pinia-orm): multiple belongsToMany relations still wrong saved#373
CodeDredd merged 1 commit intomasterfrom
pinia-orm-363

Conversation

@CodeDredd
Copy link
Copy Markdown
Owner

@CodeDredd CodeDredd commented Sep 20, 2022

🔗 Linked issue

closes #363

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Last fix didn't resolve the issue correctly. The tests were also wrong.

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@CodeDredd CodeDredd added the bug Something isn't working label Sep 20, 2022
@CodeDredd CodeDredd self-assigned this Sep 20, 2022
@CodeDredd CodeDredd merged commit 6fd7348 into master Sep 20, 2022
@CodeDredd CodeDredd deleted the pinia-orm-363 branch September 20, 2022 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

multiple belongsTo relationships between 2 entities lead to unintended outcome

1 participant