search

boboldehampsink / reverserelations

DEPRECATED - Reverse Relations Plugin for Craft CMS
MIT License
16 stars 3 forks source link

Deleting a reverse relation doesn't work #6

Open timkelty opened 8 years ago

timkelty commented 8 years ago

Adding works, but removing does not.

brodieaustin commented 8 years ago

Seconded this. I'm seeing the same problem.

chasegiunta commented 8 years ago

@timkelty @brodieaustin Opened a PR for delete functionality. Please test with your setups if you get a chance.

timkelty commented 8 years ago

@ChaseGiunta Just getting back into this...sounds like maybe there is still problems with your PR?

chasegiunta commented 8 years ago

@timkelty Yeah. Not sure if I'll ever revisit it (at least before Craft 3) but left it open in case you, @boboldehampsink, or someone else wanted to take a swing at it. Seems doable, and it would work if the Reverse Relations field was included on every form submit (like it is in the CP), but omitting the field in a front-end form submission would clear it out if it's not included.

timkelty commented 8 years ago

Got it, thanks. I’ll take a look.

On Thu, Oct 13, 2016 at 11:36 AM, Chase Giunta notifications@github.com wrote:

@timkelty https://github.com/timkelty Yeah. Not sure if I'll ever revisit it (at least before Craft 3) but left it open in case you, @boboldehampsink https://github.com/boboldehampsink, or someone else wanted to take a swing at it. Seems doable, and it would work if the Reverse Relations field was included on every form submit (like it is in the CP), but omitting the field in a front-end form submission would clear it out if it's not included.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/boboldehampsink/reverserelations/issues/6#issuecomment-253550645, or mute the thread https://github.com/notifications/unsubscribe-auth/AABHmfGoiLzf4F0kWNVQQ1i851W_wjFqks5qzk_9gaJpZM4H_Pxs .

Tim Kelty 616-334-1802