Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Localization is not managed in sendEmail process #32

Closed
finkinfridom opened this issue Feb 28, 2023 · 4 comments · Fixed by #31
Closed

Localization is not managed in sendEmail process #32

finkinfridom opened this issue Feb 28, 2023 · 4 comments · Fixed by #31

Comments

@finkinfridom
Copy link
Contributor

As per #31 , the locale info is not used when querying forms collections. This may lead to scenarios where a missing field on a localized entity is throwing errors but is impossible to spot.
@finkinfridom
Copy link
Contributor Author

If somebody faces the same issue, I published a forked version of this plugin (https://www.npmjs.com/package/@finkinfridom/payload-plugin-form-builder) with the above mentioned fixes.

@jacobsfletch
Copy link
Member

@finkinfridom I gave a review on #31 just now 👍

@jacobsfletch jacobsfletch linked a pull request May 11, 2023 that will close this issue
fix(locale): added 'locale' information in payload.findByID #31
Merged
@jacobsfletch
Copy link
Member

This was just published in 1.0.14-canary.0 🎉

@jacobsfletch
Copy link
Member

And now 1.0.14 🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(locale): added 'locale' information in payload.findByID finkinfridom/plugin-form-builder
2 participants
@finkinfridom @jacobsfletch