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

XML Sending and custom field mapping #52

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Conversation

JeffersonBledsoe
Copy link
Member

@JeffersonBledsoe JeffersonBledsoe commented Nov 23, 2022

Requires collective/collective.volto.formsupport#22

This PR adds supports for attaching an XML file containing the data submitted in the form to the email. This feature is available in collective.easyform and may be useful to those transitioning to Volto. To further help transition collective.easyform users who may be reliant on specific field IDs when using collective.easyform, the ability to customise the label for sending/ downloading XML and CSV was also added.

The below video demonstrates the custom value mapping when downloading the CSV file. The XML after the video was the contents of the file attached to the email sent when submitting the form.

Screen.Recording.2022-11-23.at.7.33.22.pm.mov
<?xml version='1.0' encoding='utf-8'?><form><field name="Custom field label">My value</field></form>

@JeffersonBledsoe JeffersonBledsoe changed the title XML Sending and custom field mapping WIP: XML Sending and custom field mapping Nov 28, 2022
@JeffersonBledsoe
Copy link
Member Author

JeffersonBledsoe commented Nov 28, 2022

Still a few issues with this PR (namely attachment fields break the form). I'll see if I can add some tests to check the XML behaviour :)

@JeffersonBledsoe JeffersonBledsoe changed the title WIP: XML Sending and custom field mapping XML Sending and custom field mapping Mar 7, 2023
@JeffersonBledsoe
Copy link
Member Author

Hey @pnicolli, any chance of a review on this?
Not sure who I should request but I've noticed you've merged the last couple. A few PRs I've got going to add easyform functionality to formblock so expect a few more reviews!

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

Successfully merging this pull request may close these issues.

2 participants