New webhook: Contact Company Subscription Change

[kuzmany]

Please be sure you are submitting this against the staging branch.

Q	A
Bug fix?
New feature?
Automated tests included?
Related user documentation PR URL
Related developer documentation PR URL
Issues addressed (#s or URLs)
BC breaks?
Deprecations?

Description:

New webhook event send when contact's companies are changed (added/removed).

Response return

• added (true or false in case if company is removed from contact)
• contact object
• company object in this event (added/removed)

Steps to test this PR:

1. Load up this PR
2. Create webhook with Contact Company Subscription Change Event (test endpoint could be take from webhook.site)
   
3. Try create contact + assign/remove companies
4. Check webhook.site if your response contains

• added (true or false if removed)
• contact
• company

List deprecations along with the new alternative:

List backwards compatibility breaks:

[npracht]

Hi there!
It has been decided to not create any extra Mautic 2 minor versions (which means no more features in M2) knowing that Mautic 3 is coming very soon.

We now want to integrate your contribution in the Mautic 3 roadmap as 3.1.0 candidate.

How to do?

1. Check if your feature or enhancement is still missing and relevant in Mautic 3
2. If Yes: rebase your PR against the 3.x branch
3. If No: please consider closing your PR if you are absolutely sure that the feature has made it into Mautic 3 already

Please report results by commenting on your PR to make us administration easier.

---

You can more information on how to do all of that on this blog post "Getting you PR ready for Mautic 3".

[kuzmany]

Rebsed

[TravisBuddy]

Travis tests have failed

Hey @kuzmany,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

TravisBuddy Request Identifier: 05bd4c50-7f4e-11ea-b434-f745fa9560ca

[mabumusa1]

I tested this PR, it works as expected I got a webhook on company change and it included the company that got added correctly to the contact

[RCheesley]

Tested and works great - adding and removing companies - and the content includes the information about both the company and the contact.

Thanks for the PR @kuzmany 🙇

[RCheesley]

Marking this as RTC subject to the checks being passed successfully - I have a suspicion it might possibly need tests but we'll see!

[codecov]

Codecov Report

Merging #8218 into staging will increase coverage by 0.01%.
The diff coverage is 76.92%.

@@              Coverage Diff              @@
##             staging    #8218      +/-   ##
=============================================
+ Coverage      29.89%   29.90%   +0.01%     
- Complexity     33278    33281       +3     
=============================================
  Files           1938     1938              
  Lines         115242   115255      +13     
=============================================
+ Hits           34447    34463      +16     
+ Misses         80795    80792       -3     

Impacted Files	Coverage Δ	Complexity Δ
...les/LeadBundle/EventListener/WebhookSubscriber.php	70.31% <76.92%> (+1.68%)	12.00 <3.00> (+3.00)
...undles/LeadBundle/Event/LeadChangeCompanyEvent.php	81.25% <0.00%> (+37.50%)	7.00% <0.00%> (ø%)

[dennisameling]

@kuzmany I just added a test for this PR, if Travis/Codecov pass this is RTC 👍

[mautibot]

This pull request has been mentioned on Mautic Community Forums. There might be relevant details there:

https://forum.mautic.org/t/mautic-3-1-new-beginnings/15787/1