-
-
Notifications
You must be signed in to change notification settings - Fork 824
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
Replace Contact Summary Events tab with Admin UI Search Display and Form #29570
Conversation
Can one of the admins verify this patch? |
1 similar comment
Can one of the admins verify this patch? |
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
Jenkins add to whitelist |
jenkins test this please |
9dd904e
to
045481d
Compare
Ok I've reviewed, tested & cleaned this up, using a linksProvider callback to give the multiple register + credit card buttons. |
Isn't there a use case for an admin transferring or cancelling a registration if someone calls or shows up at the front desk? |
Checks permissions and conditionally adds a credit card registration link
I don't know what purpose this serves, but it was added in civicrm#10695 so this keeps the new screen true to the old one.
045481d
to
9dc35dc
Compare
@guyiac @eileenmcnaughton ok I have added the self-serve link back in. |
This introduces a little regression. Someone can review it? #30086 |
Overview
Replace Contact Summary Events tab with Admin UI Search Display and Form per Coleman's request
Before
Standard Events Tab
After
Events tab of Contact Summary is a Search Display in a FormBuilder Form
Technical Details
Pretty straightforward
Comments
First commit ever. Be kind.