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.

Sign up for GitHub

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

Jump to bottom

Add a failing test for hasMany with a link #5235

Closed
wants to merge 1 commit into from
Closed

Add a failing test for hasMany with a link #5235

wants to merge 1 commit into from

Conversation

ppcano
Copy link
Contributor

@ppcano ppcano commented Oct 19, 2017

Tests for #5219

A hasMany relationship with a link will trigger the link request even if an inverse related object is pushed to the store

This was introduced at 2.14

@ppcano ppcano mentioned this pull request Oct 19, 2017
Closed
@BryanCrotaz
Copy link
Contributor

See if PR #5257 fixes this

@ppcano
Copy link
Contributor Author

ppcano commented Nov 13, 2017

@BryanCrotaz Awesome, it fixes it.

@ppcano ppcano mentioned this pull request Nov 13, 2017
Closed
@BryanCrotaz
Copy link
Contributor

BryanCrotaz commented Nov 13, 2017 •
edited by sly7-7
Loading

This test added to PR #5257

@runspired runspired mentioned this pull request Apr 3, 2018
Merged
@bmac bmac closed this in #5410 Apr 6, 2018
@runspired runspired added 🏷️ test This PR primarily adds tests for a feature and removed test-contribution labels Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
🏷️ test This PR primarily adds tests for a feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ppcano @BryanCrotaz @bmac @runspired