Skip to content
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

`gpnf-set-created-by-property-on-child-entries-after-user-registratio… #900

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sbassah
Copy link
Contributor

@sbassah sbassah commented Sep 30, 2024

…n.php` : Added new snippet for GP Nested Forms that sets the created by property on the child entries after user account is created.

Context

⛑️ Ticket(s): https://secure.helpscout.net/conversation/2719134672/71947/

Summary

When a nested form is submitted by a non-logged-in user, the created by property is not set on both the parent form entry and the child entries. However, when a, create user feed, is on the parent form to create a user account, it sets the created by property on the parent form but doesn't set that of the child form entries. This snippet sets the created by property on the child form entries, that is linked to the user registration parent form.

…n.php` : Added new snippet for GP Nested Forms that sets the created by property on the child entries after user account is created.
Copy link

github-actions bot commented Sep 30, 2024

Fails
🚫

Pull request title does match the correct format. The Pull Request title should match our Snippet Library Pull Request Title Guidelines in Notion.

🚫 Pull request title needs to end in a period or exclamation.
🚫

Commit message 'gpnf-set-created-by-property-on-child-entries-after-user-registration.php : Added new snippet for GP Nested Forms that sets the created by property on the child entries after user account is created.' does match the correct format. See our Snippet Library Commit Messages Guidelines in Notion.

Generated by 🚫 dangerJS against 2322a3c

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

Successfully merging this pull request may close these issues.

1 participant