search

spatie / laravel-newsletter

Manage Mailcoach and MailChimp newsletters in Laravel
https://freek.dev/440-easily-integrate-mailchimp-in-laravel-5
MIT License
1.6k stars 233 forks source link

Merge Fields are wrong in Readme #22

Closed gerardo15 closed 8 years ago

gerardo15 commented 8 years ago

I thought there was a problem but then after going to the mailchimp docs, i saw that first name is FNAME and last name is LNAME

Newsletter::subscribe('rincewind@discworld.com', ['firstName'=>'Rince', 'lastName'=>'Wind']);

I don't know if you did this in purpose, like everybody just put the variables they want to send, but if not, you can save people some time by adding FNAME instead of firstName and LNAME instead of lastName in the Readme

Greetings!

freekmurze commented 8 years ago

Hi,

I'm so sorry that you lost time with this. I've added a little note in the readme to make things more clear. Feel free to submit a PR to the readme if you feel it could be improved more.