Release 1819
Trello card
Context
We are removing the legacy, direct CRM integration code from the app now that we are happy that the API integration is working as expected. This will involve a number of PRs to incrementally remove the old code.
Changes proposed in this pull request
Remove legacy code from various controllers, models and jobs.
Guidance to review
There are still a few core models which use the feature flags; I'm going to address these in a separate PR as they break a bunch of outer controller/request/feature specs due to needing new mocking in place.
Half a dozen of the selenium/JS cucumber specs are failing in CI; I have no idea why as they pass locally. So I can crack on I've disabled them for now (a number of them have coverage in Capybara anyway). I'll create a ticket to come back to them later.