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

Move financial acls for membership to extension #22677

Merged
merged 2 commits into from
Feb 8, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Break out from #22499 which got stalled on an issue - not sure if this passes in isolation

Before

What is the old user-interface or technical-contract (as appropriate)?
For optimal clarity, include a concrete example such as a screenshot, GIF (LICEcap, SilentCast), or code-snippet.

After

What changed? What is new old user-interface or technical-contract?
For optimal clarity, include a concrete example such as a screenshot, GIF (LICEcap, SilentCast), or code-snippet.

Technical Details

If the PR involves technical details/changes/considerations which would not be manifest to a casual developer skimming the above sections, please describe the details here.

Comments

Anything else you would like the reviewer to note

@civibot
Copy link

civibot bot commented Feb 1, 2022

(Standard links)

@civibot civibot bot added the master label Feb 1, 2022
@eileenmcnaughton eileenmcnaughton force-pushed the aclmem branch 2 times, most recently from 37f8011 to 9dece7f Compare February 2, 2022 21:58
@eileenmcnaughton eileenmcnaughton force-pushed the aclmem branch 2 times, most recently from eab5446 to 842a480 Compare February 3, 2022 02:09
@colemanw
Copy link
Member

colemanw commented Feb 8, 2022

This is passing tests now and looks like good cleanup.

@colemanw colemanw merged commit 56102d7 into civicrm:master Feb 8, 2022
@colemanw colemanw deleted the aclmem branch February 8, 2022 17:14
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