Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

Compatibility A&M forms #8

Closed
nealzie opened this issue Jan 8, 2016 · 9 comments
Closed

Compatibility A&M forms #8

nealzie opened this issue Jan 8, 2016 · 9 comments

Comments

@nealzie
Copy link

nealzie commented Jan 8, 2016

Loving the plugin so far, I have a little issue but it might be too specific..

When I have the SEO plugin activated, sending forms through the a&m forms plugin doesn't work anymore, it gives the error.

screen shot 2016-01-08 at 19 29 21

Although this is terribly specific to another plugin, I think it ties in to a bigger issue that involves date() fields.

@khalwat
Copy link
Collaborator

khalwat commented Jan 8, 2016

huh, that's very strange. I will install it and have a look. Have you mentioned this to the author of A&M forms?

@khalwat
Copy link
Collaborator

khalwat commented Jan 8, 2016

So it's AFTER you fill out a front-end form and click the Submit button that you see this error?

@khalwat
Copy link
Collaborator

khalwat commented Jan 8, 2016

Hmmm, I think I see what's going on actually. I will have a fix for your shortly.

@khalwat
Copy link
Collaborator

khalwat commented Jan 8, 2016

Okay, I checked in a fix for you:

https://github.com/khalwat/seomatic

Let me know how that works for you.

@nealzie
Copy link
Author

nealzie commented Jan 9, 2016

Thanks for the quick help! I reinstalled the plugin with the updated version but the exact same error appears. Since it's a date() related error, I thought I'd paste another error I get with the SEO plugin which also seems to be date() related. I have a structure section with a date field and in 'demoed' I can't view any entries, I just get this:
screen shot 2016-01-09 at 11 21 57
(http://multiniels.nl/rubicon/evenementen/sharepoint-2016-de-evolutie-van-sharepoint-1)

@khalwat
Copy link
Collaborator

khalwat commented Jan 9, 2016

hmmmm. Sorry to have you do this, but can you try a change I just checked in (to address the first problem again):

https://github.com/khalwat/seomatic

@nealzie
Copy link
Author

nealzie commented Jan 9, 2016

No apologies needed! I think you broke the entire thing now though.. :D

Missing argument 2 for Craft\TemplatesService::renderObjectTemplate(), called in ......craft/plugins/seomatic/twigextensions/SeomaticTwigExtension.php on line 167 and defined

@khalwat
Copy link
Collaborator

khalwat commented Jan 9, 2016

Ugh. Okay, I pushed it again. Hopefully this will do it. Let me know...

@khalwat
Copy link
Collaborator

khalwat commented Jan 9, 2016

Looks like we got it fixed. Thanks for your patience.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants