Closed robbieaverill closed 7 years ago
@robbieaverill - I believe I have created the release and the branch as you requested, both great suggestions. Please let me know if the pre-requisites have been met now and I will go ahead and do the merge. Thanks!
@andrewvt yeah nice! Good from me now.
Resolves #12
This pull request updates the module for SilverStripe 4 compatibility.
Pre-requisites
composer.json
is aliasing these changes as2.0.x-dev
expecting that you will release the current version as 1.0.0, but if you don't, the alias should be updated to1.0.x-dev
instead)1
or similar for maintaining the current 3.x compatible version going forward (see above)Changes
BluehouseGroup\Pardot
namespace to all classes (except the Pardot class in the thirdparty dir!)BH_PARDOT_DIR
constant to represent the installation folder instead of hard-coding itNotes/to-do
The "A" and "B" are not inline in that screenshot either. Could probably be some sort of Bootstrap inline label or something that could be applied to it.
I haven't updated the screenshots
I've tested this against
4.0.0-alpha4
.