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

Injecting objects and fields into forms and columns into reports #299

Open
wants to merge 18 commits into
base: 5.x
Choose a base branch
from

Conversation

volha-pivavarchyk
Copy link

@volha-pivavarchyk volha-pivavarchyk commented Feb 6, 2023

Q A
Bug fix? (use the a.b branch) [ ]
New feature/enhancement? (use the a.x branch) [ ]
Deprecations? [ ]
BC breaks? (use the c.x branch) [ ]
Automated tests included? [ ]
Related user documentation PR URL mautic/mautic-documentation#...
Related developer documentation PR URL mautic/developer-documentation#...
Issue(s) addressed Fixes #...

⚠️ This PR needs changes from PRs #298, #308, #316 so it can be tested after merging them.

Description:

This PR injects objects and fields into the form field mapping form and object columns into the form result report.

Steps to test this PR:

  1. Open this PR for testing locally (see docs on testing PRs here)
  2. Setup a custom object. Use Mautic v5.
  3. Create a custom object and add new items.
  4. Create a form with fields mapped to the custom object. Add fields with text and select types.

image

  1. Ensure that mapping works well.
  2. Submit the form.
  3. Create a new report with the form result as a data source.
    image
  4. Check if all object columns were added.
    image
  5. Select the fields.
  6. Save rthe eport.
  7. Check if the repotr is working correctly.

@volha-pivavarchyk volha-pivavarchyk changed the base branch from staging to 5.x May 16, 2023 14:45
@volha-pivavarchyk volha-pivavarchyk changed the title WIP: Injecting objects and fields into forms and columns into reports Injecting objects and fields into forms and columns into reports May 17, 2023
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.

1 participant