-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
Update Owner Membership ID label in reports to be Primary Membership … #14585
Update Owner Membership ID label in reports to be Primary Membership … #14585
Conversation
(Standard links)
|
@seamuslee001 thanks for taking time to do this PR it really helps from a user perspective to keep things consistent. Would it be possible to move the last two options in the drop-down filter to the top of the options list? So that these "Primary Members..." options are at the top. |
@seamuslee001 you might also want to fix the spelling mistake for this PR. "Memebrship" is spelt "Membership" (but we all know what you mean) Still pretty funny! :) |
9a202b7
to
8db9dd0
Compare
ok @jusfreeman i have made both changes now would appreciate thoughts from others |
8db9dd0
to
78f3fd5
Compare
I agree with this primary mebership label chagne ;) |
@seamuslee001 It works as expected and the order is also fixed. Can you please squash the commits so that I can merge? |
…to be more consistant with rest of system Re-order owner_membership_id field options so that primary members only and non-primary-members-only filters are at the top
78f3fd5
to
455a0d0
Compare
done @yashodha |
Looks good to me - @yashodha I'll leave you to do a final check & merge |
Awesome this looks great! |
@agh1 @jusfreeman @seamuslee001 great, merging this now. |
Thanks very much, everyone! I think this is a good step forward to improving the user experience, making CiviCRM easier to understand. @yashodha @seamuslee001 @eileenmcnaughton @MegaphoneJon @agh1 |
FYI too for @jaapjansma |
Yes - good collaboration |
…to be more consistant with rest of system
Overview
This changes the label in membership reports from Owner Membership ID to be Primary Membership as per discussion on #14530
Before
Unusual label Owner Membership ID used
After
More standardised label Primary Membership used
ping @jusfreeman @MegaphoneJon @agh1 @eileenmcnaughton @yashodha i tend to agree with Andrew that Primary Membership will suffice especially given the other places where we use the term Primary rather than Owner Membership or similar