Skip to content

Comments

Set pivot data when initially syncing the relationship#739

Merged
LukeTowers merged 2 commits intodevelopfrom
wip/fix-relationcontroller
Oct 26, 2022
Merged

Set pivot data when initially syncing the relationship#739
LukeTowers merged 2 commits intodevelopfrom
wip/fix-relationcontroller

Conversation

@LukeTowers
Copy link
Member

Also performs the sync with model instances rather than just raw IDs to support relationships with a custom relatedKey set (the key on the related model's table that is stored on the pivot table to connect the relationship, normally just the primary key but can be anything).

Also performs the sync with model instances rather than just raw IDs to support relationships with a custom relatedKey set (the key on the related model's table that is stored on the pivot table to connect the relationship, normally just the primary key but can be anything).
@LukeTowers LukeTowers added maintenance PRs that fix bugs, are translation changes or make only minor changes needs review Issues/PRs that require a review from a maintainer hacktoberfest-acccepted PRs that are accepted by the maintainers as valid Hacktoberfest entries labels Oct 24, 2022
@LukeTowers LukeTowers added Status: Completed and removed needs review Issues/PRs that require a review from a maintainer hacktoberfest-acccepted PRs that are accepted by the maintainers as valid Hacktoberfest entries labels Oct 26, 2022
@LukeTowers LukeTowers added this to the v1.2.2 milestone Oct 26, 2022
@LukeTowers LukeTowers merged commit e973936 into develop Oct 26, 2022
@LukeTowers LukeTowers deleted the wip/fix-relationcontroller branch October 26, 2022 07:24
LukeTowers added a commit that referenced this pull request Oct 26, 2022
* develop:
  Set pivot data when initially syncing the relationship (#739)
  Added CLI signature to mix:list command
  Fix Navigation Manager unit test
LukeTowers added a commit that referenced this pull request Oct 26, 2022
* develop:
  Set pivot data when initially syncing the relationship (#739)
  Added CLI signature to mix:list command
  Fix Navigation Manager unit test
  Update jobs tables to support Laravel 9 (#730)
  Add migrate to list of protected commands (#733)
  Add "Send password reset email" button to backend users update page (#723)
  Adjust descriptor for local event
  Change event doc to test new event functionality in Docs plugin
  Removed NPX from webpack bin call (#728)
  Add test case for getParentData method in AJAX framework
  Recompile Snowboard
  Allow for custom AJAX error responses to be passed through handlers
  Allow detached AJAX request to be called with 2 params
  Improve IDE knowledge of the PluginBase object
  Fix support for data-request-parent
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance PRs that fix bugs, are translation changes or make only minor changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants