Cannot create a user in a contacts list

[mailjet]

Cannot create a user in a contacts list

[EtienneDepaulis]

Looks like the problem could be coming from the fact that if I do (after having tried at least once to save the contact):

contact = Mailjet::Contactslist_managecontact.new(contact_params)
contact.class.properties

I get: {}
Meaning that https://github.com/mailjet/mailjet-gem/blob/master/lib/mailjet/resource.rb#L235 then erase all the properties while formating the payload...

[WeshGuillaume]

Hi @EtienneDepaulis !
thank you for using Mailjet ! I faced the issue a few days ago! strangely, it doesn't happen without rails. I will investigate and get back to you ! Thank you !
Best,
Guillaume

[EtienneDepaulis]

Here is the dirty fix I found:

contact_params = {id: main_contact_list_id, action: "addnoforce", email: user.email}
contact = Mailjet::Contactslist_managecontact.create(contact_params)

Mailjet::Contactslist_managecontact.properties = [:email, :name, :action, :properties]

Mailjet::Contactdata.create(:contact_id => contact.contact_id, :data => [...])

[WeshGuillaume]

Hahah ! dirty indeed ! I guess you could use that until I come up with a solution !

[EtienneDepaulis]

I had too, looks like Mailjet support don't give a shit about support request sadly :/

[WeshGuillaume]

Sorry you have this feeling.
we're working hard on this issue. I manage to identify the bug. I'll keep you up to date on that !
Best,
Guillaume

[WeshGuillaume]

I found out why it didn't work. A naming conflict in our wrapper. Could you tell me if this works for you on the rails_fix: branch ?

[EtienneDepaulis]

Thanks for looking into it.
However, we already lost 3 hours today (and few more 2 weeks ago) with this bug.
This gem should really have specs in my opinion ...

[WeshGuillaume]

Hey !
I started implementing specs to test the wrapper. Thank you for your time. If you need anything else, I would be happy to help, otherwise, can you please close the issue ?
Best,
Guillaume

[EtienneDepaulis]

Has this gem been updated on Rubygems ?
If not, the issue still exists ...

[WeshGuillaume]

Hey ! @EtienneDepaulis

I am just letting you know that we released a new version on RubyGem with specs this time, and the rails_fix branch is now merged into master. We are planning to constantly grow our tests over time. Feel free to contribute if you think it lacks some useful checks !

Best,
Guillaume from Mailjet !

[EtienneDepaulis]

awesome thx :)