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

Afform - Add table of submission results #22009

Merged
merged 1 commit into from
Nov 19, 2021

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Nov 8, 2021

Overview

Uses a SearchDisplay in an Afform to display form submissions.

Before

No way to view Afform submissions

After

image

Technical Details

This demonstrates the new capabilities of ManagedEntity to package a search.
Takes advantage of the new 'unmodified' update mode which allows searches to be locally overridden, upgraded, and reverted. Requires #21989

Comments

As of this PR, SearchKit is now required by the AfformAdmin extension.

@civibot
Copy link

civibot bot commented Nov 8, 2021

(Standard links)

@civibot civibot bot added the master label Nov 8, 2021
@seamuslee001
Copy link
Contributor

code seems fine to me but need to do an r-run

@colemanw colemanw marked this pull request as draft November 9, 2021 20:24
@colemanw colemanw force-pushed the afformSubmissionTable branch from 8713250 to 5faf958 Compare November 10, 2021 21:55
Uses a searchDisplay in an Afform to display form submissions.
Takes advantage of the new 'unmodified' update mode.
@colemanw colemanw force-pushed the afformSubmissionTable branch from 5faf958 to cb80705 Compare November 10, 2021 22:00
@colemanw colemanw marked this pull request as ready for review November 13, 2021 03:19
@seamuslee001
Copy link
Contributor

This looks good to me merging

@seamuslee001 seamuslee001 merged commit cf9234b into civicrm:master Nov 19, 2021
@seamuslee001 seamuslee001 deleted the afformSubmissionTable branch November 19, 2021 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants