search

klaviyo / php-klaviyo

PHP wrapper for the Klaviyo API
59 stars 48 forks source link

Is this project auto generated? #44

Closed loevgaard closed 3 years ago

loevgaard commented 3 years ago

Is this project auto generated? If not I will add some PRs improving the overall quality and developer experience if that would be okay?

remstone7 commented 3 years ago

It's not autogenerated, what changes are you looking to make?

loevgaard commented 3 years ago

Lots

What do you think? :)

remstone7 commented 3 years ago

all of that looks good we'd definitely appreciate it!

For php 7.4, would you be making this the minimum requirement or just expanding to support it? For the ci, what would you be using?

loevgaard commented 3 years ago

Here is a first PR: https://github.com/klaviyo/php-klaviyo/pull/45

remstone7 commented 3 years ago

okay cool 👍 will review on the pr's, closing this issue!