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

CRM-16000 Member Role Sync: precedence on role grant/revoke #316

Closed
wants to merge 1 commit into from

Conversation

agh1
Copy link
Contributor

@agh1 agh1 commented Sep 15, 2015

Simply sort the memberships by status weight descending


Simply sort the memberships by status weight descending

----------------------------------------
* CRM-16000: Member Role Sync Uses Inactive Memberships Over Active Memberships
  https://issues.civicrm.org/jira/browse/CRM-16000
@agh1
Copy link
Contributor Author

agh1 commented Sep 15, 2015

replaces #315

@totten
Copy link
Member

totten commented May 21, 2016

(Note: Comments from previous review cycles are in JIRA.)

@JoeMurray
Copy link

Andrew, as Release Manager this month, I'm trying to recruit people to help pare down the backlog of almost 100 PRs, some going back to last summer (like this one). I'm wondering if you would be able to help QA another PR if I got someone to QA this PR?

@mlutfy
Copy link
Member

mlutfy commented Oct 26, 2018

Closing this PR, since it has conflicts and is fairly old, without recent activity. It seems like a valid use-case. If someone is still running into this issue, can you open a gitlab issue?

@mlutfy mlutfy closed this Oct 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants