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

Update Owner Membership ID label in reports to be Primary Membership … #14585

Merged

Conversation

seamuslee001
Copy link
Contributor

…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

@civibot civibot bot added the master label Jun 18, 2019
@civibot
Copy link

civibot bot commented Jun 18, 2019

(Standard links)

@jusfreeman
Copy link
Contributor

jusfreeman commented Jun 19, 2019

@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.

@jusfreeman
Copy link
Contributor

jusfreeman commented Jun 19, 2019

@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! :)

@seamuslee001 seamuslee001 force-pushed the primary_mebership_label_chagne branch from 9a202b7 to 8db9dd0 Compare June 19, 2019 00:55
@seamuslee001 seamuslee001 changed the title Update Owner Membership ID label in reports to be Primary Memebrship … Update Owner Membership ID label in reports to be Primary Membership … Jun 19, 2019
@seamuslee001
Copy link
Contributor Author

ok @jusfreeman i have made both changes now would appreciate thoughts from others

@seamuslee001 seamuslee001 force-pushed the primary_mebership_label_chagne branch from 8db9dd0 to 78f3fd5 Compare June 19, 2019 01:05
@jusfreeman
Copy link
Contributor

I agree with this primary mebership label chagne

;)

@yashodha
Copy link
Contributor

@seamuslee001 It works as expected and the order is also fixed.
mem_primary

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
@seamuslee001 seamuslee001 force-pushed the primary_mebership_label_chagne branch from 78f3fd5 to 455a0d0 Compare June 19, 2019 10:02
@seamuslee001
Copy link
Contributor Author

done @yashodha

@eileenmcnaughton
Copy link
Contributor

Looks good to me - @yashodha I'll leave you to do a final check & merge

@agh1
Copy link
Contributor

agh1 commented Jun 19, 2019

Awesome this looks great!

@yashodha
Copy link
Contributor

@agh1 @jusfreeman @seamuslee001 great, merging this now.

@yashodha yashodha merged commit c70f3fb into civicrm:master Jun 19, 2019
@eileenmcnaughton eileenmcnaughton deleted the primary_mebership_label_chagne branch June 19, 2019 17:23
@jusfreeman
Copy link
Contributor

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

@jusfreeman
Copy link
Contributor

FYI too for @jaapjansma

@eileenmcnaughton
Copy link
Contributor

Yes - good collaboration

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.

5 participants