Skip to content

Add Attributes to the Pivot Embed #1796

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

Closed
wants to merge 23 commits into from
Closed

Add Attributes to the Pivot Embed #1796

wants to merge 23 commits into from

Conversation

AAllport
Copy link

@AAllport AAllport commented Jul 31, 2019

Adds Attributes to the Pivot Embed,

Currently got the attribute saving, and retrieval.
Just having a few issues when it comes to eager loading

Help would be appreciated

Supersedes: #1794
Fixes: #1618

@AAllport
Copy link
Author

AAllport commented Aug 1, 2019

Current issue is for RelationsTest.testBelongsToManyCustom()
Loading the relationship with custom param's causes breakage, and I could do with some test's to legacy

  • [Fix] RelationsTest.testBelongsToManyCustom()
  • [Test] Legacy relations

@AAllport
Copy link
Author

AAllport commented Aug 2, 2019

Currently seeing this error on Cadacy
http://phpmd.org/rules/cleancode.html#staticaccess

I believe this is safe to ignore

@AAllport AAllport marked this pull request as ready for review August 2, 2019 14:30
@AAllport AAllport changed the title Add Attributes to the Pivot Embed [WIP] Add Attributes to the Pivot Embed Aug 2, 2019
@AAllport
Copy link
Author

AAllport commented Aug 2, 2019

Please review: @jenssegers

@AAllport AAllport changed the title [WIP] Add Attributes to the Pivot Embed Add Attributes to the Pivot Embed Aug 5, 2019
@Smolevich
Copy link
Contributor

@AAllport, please resolve conflicts

@divine
Copy link
Contributor

divine commented Feb 16, 2020

Closing in favor of #1952, this hasn't been even replied or taken care at all.

@divine divine closed this Feb 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The withPivot fields are getting ignored.
3 participants