Conversation
|
Are you applying some kind of coding style? If not, I'd be willing to make the bits of code that you've finished in this PR compatible with PSR-2. I'm not really sure if I can just pick the code from the branch |
|
@stefantalen I am not sure yet but I can ping you I get the correct structure and standard. |
|
Looks like I can create a new PR based on the branch |
|
Yeah. You can give it a try. I want to keep all the endpoints in their own classes. Thanks! — On Tue, Oct 7, 2014 at 7:35 PM, Stefan Talen notifications@github.com
|
|
Looks like a Symfony Bundle should have a recommended structure: http://symfony.com/doc/master/cookbook/bundles/best_practices.html I guess there is no need for a |
|
@stefantalen yeah the bundle was following before but they are make changes to the structure in symfony 3. Coming back from Python and Java I had been using src/ but if you see any reason not to use it I am up to changing it back. It also looks like Symfony uses it too https://github.com/symfony/symfony. |
|
@miguel250 I'm not considering the symfony/symfony as a pure bundle so I'm not sure if that applies to the structure. In the end it doesn't really matter that much, let's keep it in the |
|
Hello, i would like to know how i can contribute to this bundle. i really need to delete static segments. i tried to override the bundle by adding this function and it' s working fine /**
* delete segment by ID
* @return bool
*/
public function listStaticSegmentDel($seg_id)
{
$payload = array(
'id' => $this->listId,
'seg_id' => $seg_id,
);
$apiCall = 'listStaticSegmentDel';
$data = $this->makeRequest($apiCall, $payload);
return $data;
}
i finally found the right URLs for all methodes http://apidocs.mailchimp.com/api/1.3/#listrelated |
|
i have most of the methods needed implemented well in your bundle, tell me if i can contribute to your work. thanks |
|
@miguel250 The |
|
@stefantalen here says you can ignore it for libraries if you want since it doesn't do anything. https://getcomposer.org/doc/02-libraries.md#lock-file |
|
@miguel250 Makes sense 👍 |
|
I kinda need this bundle, I was wondering if v2 is ready yet. I mean last time something updated was 28 days ago! |
|
Yeah would love to use the v2 version too, at least be able to subscribe to a list (and unsubscribe). Anyway we could help otherwise? |
|
Guys Needs Help for this one? |
Fix: #33
Close: #25