MZMailChimpBundle icon indicating copy to clipboard operation
MZMailChimpBundle copied to clipboard

Start support MailChimp API v2.

Open miguel250 opened this issue 10 years ago • 15 comments

  • [x] Create new directory structure
  • [ ] Check for breaking changes in Symfony 3
  • [ ] Silex support
  • [ ] Stop using Buzz client
  • [ ] Every method should have a test case and make a request to the MailChimp API.
  • [ ] Issue #26
  • [ ] New methods
    • [ ] Single campaign (#27)
    • [ ] createList (#32)
    • [ ] listLists (#23)
    • [ ] staticSegmentDel (#37)
  • [ ] Add support to old methods:
    • [ ] listSubscribe
    • [ ] listUnSubscribe
    • [ ] listUpdateMember
    • [ ] listInterestGroupingAdd
    • [ ] campaignCreate
    • [ ] campaignSendTest
    • [ ] campaignSendNow
    • [ ] listStaticSegmentAdd
    • [ ] listStaticSegmentMembersAdd
    • [ ] listStaticSegments
    • [ ] campaigns
    • [ ] campaignStats

Fix: #33 Close: #25

miguel250 avatar Oct 07 '14 05:10 miguel250

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 API-v2 you've created so let me know what you think.

stefantalen avatar Oct 07 '14 21:10 stefantalen

@stefantalen I am not sure yet but I can ping you I get the correct structure and standard.

miguel250 avatar Oct 07 '14 21:10 miguel250

Looks like I can create a new PR based on the branch API-v2, for the file structure you could use PSR-4 since it already kind of has that structure now in the v2-branch.

stefantalen avatar Oct 07 '14 23:10 stefantalen

Yeah. You can give it a try. I want to keep all the endpoints in their own classes.

Thanks!

— Miguel Perez

On Tue, Oct 7, 2014 at 7:35 PM, Stefan Talen [email protected] wrote:

Looks like I can create a new PR based on the branch API-v2, for the file structure you could use PSR-4 since it already kind of has that structure now in the v2-branch.

Reply to this email directly or view it on GitHub: https://github.com/miguel250/MZMailChimpBundle/pull/34#issuecomment-58280299

miguel250 avatar Oct 08 '14 00:10 miguel250

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 src folder then.

stefantalen avatar Oct 08 '14 16:10 stefantalen

@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 avatar Oct 08 '14 17:10 miguel250

@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 src-folder just to keep the focus on making this bundle work with the API

stefantalen avatar Oct 08 '14 17:10 stefantalen

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

medy36 avatar Oct 15 '14 08:10 medy36

i have most of the methods needed implemented well in your bundle, tell me if i can contribute to your work. thanks

medy36 avatar Oct 15 '14 13:10 medy36

@miguel250 The composer.lock file should not be ignored https://getcomposer.org/doc/01-basic-usage.md#composer-lock-the-lock-file

stefantalen avatar Oct 15 '14 21:10 stefantalen

@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 avatar Oct 15 '14 21:10 miguel250

@miguel250 Makes sense :+1:

stefantalen avatar Oct 16 '14 08:10 stefantalen

I kinda need this bundle, I was wondering if v2 is ready yet. I mean last time something updated was 28 days ago!

TomSchillemans avatar Nov 10 '14 12:11 TomSchillemans

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?

evertharmeling avatar Nov 11 '14 14:11 evertharmeling

Guys Needs Help for this one?

alexandretaz avatar Mar 07 '15 17:03 alexandretaz