3 ms·
Yeah "CreatesContract" is a weird name. ContractCreator, or ContractRepository (so it can do more than just create objects) makes more sense to me. Plus he sho
by bradwestness 14y ago
Yeah "CreatesContract" is a weird name. ContractCreator, or ContractRepository (so it can do more than just create objects) makes more sense to me.
Plus he should probably just break the send_email method into it's own method, which he can then call depending on if create_contact returns successfully.