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

dev/core#3778 Add 'Registered by Contact ID' (created_id) to civireport #24749

Merged
merged 1 commit into from Oct 14, 2022

Conversation

mattwire
Copy link
Contributor

@mattwire mattwire commented Oct 14, 2022

Overview

See https://lab.civicrm.org/dev/core/-/issues/3778

Add the "Registered by Contact ID" created_id field to CiviReport template.

Before

Only the old "Registered by Participant ID" available.

After

Both available.

Technical Details

Just a tweak to the report.

Comments

@civibot
Copy link

civibot bot commented Oct 14, 2022

(Standard links)

@civibot civibot bot added the master label Oct 14, 2022
@mattwire mattwire changed the title dev/core#3778 Add 'Registered by Contact ID' to civireport dev/core#3778 Add 'Registered by Contact ID' (created_id) to civireport Oct 14, 2022
@demeritcowboy
Copy link
Contributor

If this makes sense to people then ok. It's a bit confusing to me but I haven't been following the earlier PRs.

Untitled4

Untitled3

@demeritcowboy demeritcowboy added the merge ready PR will be merged after a few days if there are no objections label Oct 14, 2022
@mattwire
Copy link
Contributor Author

@demeritcowboy The new field is called "created_id". It's already available in searchkit but putting it into civireport makes it easier for existing sites to transition / start using it. Hopefully someone will comment :-)

@colemanw
Copy link
Member

Makes sense to me too.

@colemanw colemanw merged commit 8194498 into civicrm:master Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master merge ready PR will be merged after a few days if there are no objections
Projects
None yet
3 participants