EvanHerman / yikes-inc-easy-mailchimp-extender

Easy Forms for MailChimp WordPress Plugin
http://yikesplugins.com
GNU General Public License v2.0
25 stars 28 forks source link

Issues with MailChimp Word Press Plug In #82

Closed markwelsh closed 10 years ago

markwelsh commented 10 years ago

I have it installed and ready on my website, but when people put in their information I can't their information on mailchimp or wordpress dashboard. Am I missing a step?

TheTracyL commented 10 years ago

Hi Mark, Do you have a link to the form on your site?

markwelsh commented 10 years ago

Which form?

On Mon, Jun 9, 2014 at 4:22 PM, Tracy Levesque notifications@github.com wrote:

Hi Mark, Do you have a link to the form on your site?

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45539161 .

TheTracyL commented 10 years ago

The MailChimp form you added to your website using our plugin.

markwelsh commented 10 years ago

http://arkeapps.com/wp-admin/admin.php?page=yks-mailchimp-form-lists

On Mon, Jun 9, 2014 at 4:32 PM, Tracy Levesque notifications@github.com wrote:

The MailChimp form you added to your website using our plugin.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45540260 .

TheTracyL commented 10 years ago

Hi Mark.

I mean on the front-end of your site. Did you add a form to a page or post?

markwelsh commented 10 years ago

Gotcha. I added it as a secondary widget. Should I change it as part of the page?

On Mon, Jun 9, 2014 at 7:59 PM, Tracy Levesque notifications@github.com wrote:

Hi Mark.

I mean on the front-end of your site. Did you add a form to a page or post?

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45559352 .

TheTracyL commented 10 years ago

Ah, ok. So it is in the widget in the right sidebar here: http://arkeapps.com/ ?

markwelsh commented 10 years ago

yes

On Mon, Jun 9, 2014 at 8:04 PM, Tracy Levesque notifications@github.com wrote:

Ah, ok. So it is in the widget in the right sidebar here: http://arkeapps.com/ ?

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45559723 .

TheTracyL commented 10 years ago

Ok, our programmer @EvanHerman can take the troubleshooting from here. Thanks!

markwelsh commented 10 years ago

Thank you! Sorry for the delays

On Mon, Jun 9, 2014 at 8:09 PM, Tracy Levesque notifications@github.com wrote:

Ok, our programmer @EvanHerman https://github.com/EvanHerman can take the troubleshooting from here. Thanks!

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45560112 .

EvanHerman commented 10 years ago

Hey Mark,

What do you have entered in the dashboard for your success message?

Have you entered some HTML into the WYSIWYG? Can you copy and paste exactly what you have here?

Thanks, Evan

markwelsh commented 10 years ago

We promise to deliver relevant content about Arke Apps and make sure you do not consider any of this spam :)

On Mon, Jun 9, 2014 at 8:13 PM, Evan Herman notifications@github.com wrote:

Hey Mark,

What do you have entered in the dashboard for your success message?

Have you entered some HTML into the WYSIWYG?

Thanks, Evan

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45560353 .

EvanHerman commented 10 years ago

And have you included any HTML into the single and double optin message fields?

The only reason I ask is because when I check the inspector this is what I'm seeing for your single optin message: `

\n
\n\n
\n \n \n
\n

\n \n \n \n

\n
\n
Thank you for your interest in Arke Apps!

\n

We promise to deliver relevant content about Arke Apps and make sure you do not consider any of this spam :)

\n
markwelsh commented 10 years ago

Okay so I did just see I had some HTML in the single message field for the jetpack contact form in the message. I don't know if that prevented it. I just took it out. Can you check to see if that helped?

Thanks!

On Mon, Jun 9, 2014 at 8:24 PM, Evan Herman notifications@github.com wrote:

And have you included any HTML into the single and double optin message fields?

The only reason I ask is because when I check the inspector this is what I'm seeing for your single optin message:

v id='contact-form-34'>\n

\n\n
\n \n \n
\n

\n \n \n \n

\n
\n
Thank you for your interest in Arke Apps!

\n

We promise to deliver relevant content about Arke Apps and make sure you do not consider any of this spam :)

\n
EvanHerman commented 10 years ago

Hi Mark,

I'm still seeing the HTML in the field for our plugin, but I'm not sure the html would break it tho, as I just set up a local environment, with the Twenty Thirteen theme installed. I placed our widget in the secondary sidebar area and added HTML to the success message, and everything worked for me. So I'm having a hard time replicating the issue on my end.

What else do you have in the secondary sidebar? I'm trying to figure out where the sharedaddy element is coming from inside of the success message fields.

EvanHerman commented 10 years ago

Where on your site is the Jetpack contact form? I'm not seeing it displayed anywhere. Are you trying to display the Jetpack contact form inside of the success message?

markwelsh commented 10 years ago

It is in the post post week 1 reflections and then I did have it in the sign up message for the mail chimp form but i just took it out

On Mon, Jun 9, 2014 at 8:43 PM, Evan Herman notifications@github.com wrote:

Where on your site is the Jetpack contact form? I'm not seeing it displayed anywhere.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45562100 .

EvanHerman commented 10 years ago

It looks like you still have the sharedaddy code in there for both the single and double optin fields which is causing the error.

For some reason that and Jetpack are using single quotes to denote class names, which is causing the variables to escape improperly. If you take out the sharedaddy code everything should work properly as a temporary fix.

As a long term permanent solution, I'll have to take a closer look as to why all of the quotes are causing the issues, and push out an update for you first thing in the morning, which will certainly alleviate the issue so you can include your Jetpack comment form and the sharedaddy code in there.

I have a few other features such as CAPTCHA support to stop spammers, that I'd like to get included in the next update as well.

Let me know if that does in fact resolve the problem for the time being.

markwelsh commented 10 years ago

Wait so where do I take out the single quotations? or should I just take out the jetpack contact form?

On Mon, Jun 9, 2014 at 8:51 PM, Evan Herman notifications@github.com wrote:

It looks like you still have the sharedaddy code in there for both the single and double optin fields which is causing the error.

For some reason they're using single quotations in there class names, which is causing the variables to escape improperly. If you take them out everything should work properly.

I'll have to take a closer look as to why all of the quotes are causing the issues, and push out an update for you first thing in the morning.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45562550 .

EvanHerman commented 10 years ago

I would take out all elements and code temporarily except for your success message

"We promise to deliver relevant content about Arke Apps and make sure you do not consider any of this spam :)"

I know the simley face is an image, I'm fairly confident you can leave that in there.

You can't remove the single quotes, they are being generated by the Jetpack contact form shortcode. I believe that you need to remove the share daddy shortcode as well.

markwelsh commented 10 years ago

okay will do. Then will that email address be put directly into my mail chimp list?

On Mon, Jun 9, 2014 at 8:58 PM, Evan Herman notifications@github.com wrote:

I would take out all elements temporarily except for your success message

"We promise to deliver relevant content about Arke Apps and make sure you do not consider any of this spam :)"

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45562922 .

EvanHerman commented 10 years ago

It should do, yes sir. Let me know when you do all that, and I'll input my email and we can test it out.

markwelsh commented 10 years ago

I just had a friend try it and it did not work.

On Mon, Jun 9, 2014 at 9:01 PM, Evan Herman notifications@github.com wrote:

It should do, yes sir

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45563110 .

EvanHerman commented 10 years ago

You still have sharedaddy code in your WYSIWYG area, that or it's being added the it for one reason or another. The sharedaddy code should not be included in the optin message. As you can see here, it is:

`

Thank you for your interest in Arke Apps.

\n

We promise to deliver relevant content about Arke Apps and make sure you do not consider any of this spam

\n' `
markwelsh commented 10 years ago

That's odd. I'll try to play around with it. Thanks for your help. Do I need to add the short code at all?

On Mon, Jun 9, 2014 at 9:06 PM, Evan Herman notifications@github.com wrote:

You still have sharedaddy code in your WYSIWYG area, that or it's being added the it for one reason or another. The sharedaddy code should not be included in the optin message. As you can see here, it is:

Thank you for your interest in Arke Apps.

\n

We promise to deliver relevant content about Arke Apps and make sure you do not consider any of this spam

\n' — Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45563352 .
EvanHerman commented 10 years ago

If you check the post here http://arkeapps.com/?p=100 , the MailChimp plugin works in the sidebar. For whatever reason, on your homepage some Share daddy code is being generated and added to the confirmation message.

Can you try and disable Jetpack sharing? Do you know what Share daddy is at all?

markwelsh commented 10 years ago

Yes I'll try to do that. I know the MailChimp plug in is seen there but when people plug in their information they don't get a confirmation or I don't receive their information.

I will disable the jetpack sharing. Isn't share daddy just the plug in to share on social media?

On Mon, Jun 9, 2014 at 9:17 PM, Evan Herman notifications@github.com wrote:

If you check the post here http://arkeapps.com/?p=100 , the MailChimp plugin works in the sidebar. For whatever reason, on your homepage some Share daddy code is being generated and added to the confirmation message.

Can you try and disable Jetpack sharing? Do you know what Share daddy is at all?

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45563974 .

EvanHerman commented 10 years ago

I'm not really sure what is going on your end. I seem to have replicated your exact environment, activated Jetpack with social sharing and things seem to work on my end. Is it possible you can create me a temporary admin account so I can take a closer look first hand. I just can't seem to replicate the issue exactly as you have it.

Also, the reason users aren't being subscribed is because you have errors running within the form. When you hit submit, the page shouldn't be refreshing, it should submit via AJAX and you should receive the confirmation or error message without a page load. If you enter your information on that page specifically, you should be getting a confirmation email. No user will be added to any lists until they have confirmed their email.

test_mailchimp_jetpack_error

In regards to the Sharing buttons appearing in the sidebar, that shouldn't be happening. My guess is Jetpack is appending the sharing buttons to any elements with the class the_content, which gets generated whenever a WYSIWYG editor is used. I'll make the adjustment in the morning to change the class name to avoid this conflict. The sharing buttons shouldn't append to this area of the site.

markwelsh commented 10 years ago

Makes sense. I'll create a temporary admin account for you.

On Mon, Jun 9, 2014 at 9:24 PM, Evan Herman notifications@github.com wrote:

I'm not really sure what is going on your end. I seem to have replicated your exact environment, activated Jetpack with social sharing and things seem to work on my end. Is it possible you can create me a temporary admin account so I can take a closer look first hand. I just can't seem to replicate the issue exactly as you have it.

Also, the reason users aren't being subscribed is because you have errors running within the form. When you hit submit, the page shouldn't be refreshing, it should submit via AJAX and you should receive the confirmation or error message without a page load.

[image: test_mailchimp_jetpack_error] https://cloud.githubusercontent.com/assets/5321364/3224740/06683d4a-f03e-11e3-9fc0-af1ed3ebe272.png

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45564319 .

EvanHerman commented 10 years ago

I just added my email and name and received the success message on that specific page.

You should be seeing my email, first and last name over on MailChimp shortly. You may be checking the wrong form.

markwelsh commented 10 years ago

Great. I just removed all shares and likes from all pages. I guess that maybe did the trick

On Mon, Jun 9, 2014 at 9:30 PM, Evan Herman notifications@github.com wrote:

I just added my email and name and received the success message.

You should be seeing my email, first and last name over on MailChimp shortly.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45564572 .

EvanHerman commented 10 years ago

I don't think it did the trick as I'm not able to submit it on the front page, I'm still seeing that error.

markwelsh commented 10 years ago

I just got your email on Mail Chimp. So there is still an error?

On Mon, Jun 9, 2014 at 9:32 PM, Evan Herman notifications@github.com wrote:

Not on the front page, I'm still seeing that error.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45564695 .

EvanHerman commented 10 years ago

The error still exists on the front page. On the page I keep mentioning, no.

markwelsh commented 10 years ago

So I click them off on the front page as this screen shot shows, but then when I click update the boxes go back to being checked. Do you know why that is?

On Mon, Jun 9, 2014 at 9:33 PM, Evan Herman notifications@github.com wrote:

The error still exists on the front page. On the page I keep mentioning, no.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45564766 .

EvanHerman commented 10 years ago

Which screenshot? If you're referring to the Jetpack settings for Sharing, I'm not sure why that is. It generally indicates some sort of error that's preventing the plugin settings from saving.

markwelsh commented 10 years ago

Did you not get the screen shot I sent in that last email? I've been able to turn off sharing on all my other pages, but when I update the front page it defaults back to sharing every time.

On Mon, Jun 9, 2014 at 9:40 PM, Evan Herman notifications@github.com wrote:

What screenshot? If you're referring to the Jetpack settings for Sharing, I'm not sure why that is. It generally indicates some sort of error that's preventing the plugin settings from saving.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45565100 .

EvanHerman commented 10 years ago

No I didn't get a screenshot. I'm also on Github.com, not checking this via email.

markwelsh commented 10 years ago

Oh ok. But yes the issue seems to be when I uncheck the sharing boxes on the front page and then update it the boxes go back to being checked.

When I unchecked the boxes on the other posts they stay unchecked when I update so that is why you were able to get the message.

On Mon, Jun 9, 2014 at 9:46 PM, Evan Herman notifications@github.com wrote:

No I didn't get a screenshot. I'm also on Github.com, not checking this via email.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45565394 .

EvanHerman commented 10 years ago

The only thing I can recommend is disabling every plugin you have besides jetpack, and try to disable it again.

If that works, enable your other plugins.

Since you're already using the Twenty Thirteen theme it's most certainly not a theme issue. It could be a conflict with another plugin as well, that's why I recommend disabling all your other plugins to test that theory.

markwelsh commented 10 years ago

If I disable all my plug ins will it be easy to enable them can to how I have them? Just nervous to make that play.

You mentioned doing something to the code this morning. What in particular were you going to do?

Thanks

On Mon, Jun 9, 2014 at 9:50 PM, Evan Herman notifications@github.com wrote:

The only thing I can recommend is disabling every plugin you have besides jetpack, and try to disable it again.

If that works, enable your other plugins.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45565616 .

EvanHerman commented 10 years ago

I'll let you know when I have a working version for you.

On Jun 10, 2014, at 8:39 AM, markwelsh notifications@github.com wrote:

If I disable all my plug ins will it be easy to enable them can to how I have them? Just nervous to make that play.

You mentioned doing something to the code this morning. What in particular were you going to do?

Thanks

On Mon, Jun 9, 2014 at 9:50 PM, Evan Herman notifications@github.com wrote:

The only thing I can recommend is disabling every plugin you have besides jetpack, and try to disable it again.

If that works, enable your other plugins.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45565616 .

— Reply to this email directly or view it on GitHub.

markwelsh commented 10 years ago

thanks

On Tue, Jun 10, 2014 at 8:42 AM, Evan Herman notifications@github.com wrote:

I'll let you know when I have a working version for you.

On Jun 10, 2014, at 8:39 AM, markwelsh notifications@github.com wrote:

If I disable all my plug ins will it be easy to enable them can to how I have them? Just nervous to make that play.

You mentioned doing something to the code this morning. What in particular were you going to do?

Thanks

On Mon, Jun 9, 2014 at 9:50 PM, Evan Herman notifications@github.com wrote:

The only thing I can recommend is disabling every plugin you have besides jetpack, and try to disable it again.

If that works, enable your other plugins.

— Reply to this email directly or view it on GitHub < https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45565616>

.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45608068 .

markwelsh commented 10 years ago

Is there a way I can go into my editor and adjust the code?

On Tue, Jun 10, 2014 at 8:43 AM, Mark Welsh mark@arkeapps.com wrote:

thanks

On Tue, Jun 10, 2014 at 8:42 AM, Evan Herman notifications@github.com wrote:

I'll let you know when I have a working version for you.

On Jun 10, 2014, at 8:39 AM, markwelsh notifications@github.com wrote:

If I disable all my plug ins will it be easy to enable them can to how I have them? Just nervous to make that play.

You mentioned doing something to the code this morning. What in particular were you going to do?

Thanks

On Mon, Jun 9, 2014 at 9:50 PM, Evan Herman notifications@github.com wrote:

The only thing I can recommend is disabling every plugin you have besides jetpack, and try to disable it again.

If that works, enable your other plugins.

— Reply to this email directly or view it on GitHub < https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45565616>

.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45608068 .

markwelsh commented 10 years ago

Okay, I disabled JetPack only and it seems to be working.

On Tue, Jun 10, 2014 at 8:49 AM, Mark Welsh mark@arkeapps.com wrote:

Is there a way I can go into my editor and adjust the code?

On Tue, Jun 10, 2014 at 8:43 AM, Mark Welsh mark@arkeapps.com wrote:

thanks

On Tue, Jun 10, 2014 at 8:42 AM, Evan Herman notifications@github.com wrote:

I'll let you know when I have a working version for you.

On Jun 10, 2014, at 8:39 AM, markwelsh notifications@github.com wrote:

If I disable all my plug ins will it be easy to enable them can to how I have them? Just nervous to make that play.

You mentioned doing something to the code this morning. What in particular were you going to do?

Thanks

On Mon, Jun 9, 2014 at 9:50 PM, Evan Herman notifications@github.com

wrote:

The only thing I can recommend is disabling every plugin you have besides jetpack, and try to disable it again.

If that works, enable your other plugins.

— Reply to this email directly or view it on GitHub < https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45565616>

.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45608068 .

EvanHerman commented 10 years ago

There is a way, but it's not ideal as the adjustments will get overwritten in the next update.

I should have an updated version for you in the next 30 minutes that you can try out.

markwelsh commented 10 years ago

Great, thanks for all your help.

On Tue, Jun 10, 2014 at 9:08 AM, Evan Herman notifications@github.com wrote:

There is a way, but it's not ideal as the adjustments will get overwritten in the next update.

I should have an updated version for you in the next 30 minutes that you can try out.

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45610701 .

EvanHerman commented 10 years ago

Here is a link to the 4.4 alpha release. Give it a test and let me know if it is still giving you an error when using it in conjunction with jetpack and the sharing module.

https://app.box.com/s/spq4bh9ku4piadg2ircf

EvanHerman commented 10 years ago

Hi Mark,

I'm wondering if I can close out this issue, I haven't heard back from you.

Thanks, Evan

markwelsh commented 10 years ago

Yes you can close it out. Thank you for your help

On Tue, Jun 10, 2014 at 3:50 PM, Evan Herman notifications@github.com wrote:

Hi Mark,

I'm wondering if I can close out this issue, I haven't heard back from you.

Thanks, Evan

— Reply to this email directly or view it on GitHub https://github.com/yikesinc/yikes-inc-easy-mailchimp-extender/issues/82#issuecomment-45662658 .