-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecated property: 'process' #16
Comments
Hey @DarkChris86, thanks for the PRs 🙂 If you look at the docs, the signature of the functions also changed. Now they receive objects, like this: const processor = {
test: ({ node }) => { /* ... */ },
processor: ({ node }) => { /* ... */ },
}; The change is backward compatible but we will deprecate the old API at some point. Would you mind adding that change to your PRs? Thanks!! |
Hey @luisherranz . Yes, I'm going to modify them like this! Thank you for friendly note! ; ) |
Thanks to you @DarkChris86! 😄 |
It's done. Could you check them ( @luisherranz )? Thanks, |
@DarkChris86 |
@imranhsayed Thanks, |
Thanks @DarkChris86! 🙌 🙂 (and @imranhsayed) |
I'm still getting this error on the latest version installed by NPM (0.1.4) EDIT: The form works, I just needed to use a WYSIWYG field instead of a text area as provided by ACF Pro. |
@imranhsayed can you please release a new version in npm? |
@imranhsayed any update on releasing a new version in npm? The contact form isn't even showing up on my site now. It's funny, on my laptop, I don't get this error, so it might be related to the version of npm being used. I'll look into this a bit more... |
Released the new version on npm |
Thanks @imranhsayed the latest version doesn't have this bug but I have found a new error with the latest version I will create another issue about. |
Hey @imranhsayed!
I got the following message in frontity app:
Bests,
Chris
The text was updated successfully, but these errors were encountered: