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

PCHR-3545: Changed Order For Individual Prefix #2691

Conversation

backstageel
Copy link
Contributor

@backstageel backstageel commented Jun 19, 2018

Overview

Change order of individual prefix options to:

  1. Mr.
  2. Mrs.
  3. Ms.
  4. Miss
  5. Dr.

Before

The order used was the default for CiviCRM.
screenshot from 2018-06-19 15 44 05

After

screenshot from 2018-06-19 12 34 04

@backstageel backstageel changed the base branch from staging to PCHR-3536-Display-Preferences June 19, 2018 10:47
@backstageel backstageel requested a review from ajesamson June 19, 2018 11:43
@ajesamson
Copy link
Contributor

ajesamson commented Jun 19, 2018

Can you provide screenshot for the Before as someone new to the system might not know what the order was if the need arose to know why order was changed.
The overview should also read which type of options you are changing the order.

$newWeight = 2;
break;

case "Ms.":
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly replace with single quote

$newWeight = 4;
break;

case "Dr.":
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Single quote here as well


/**
* Changes The Order of Individual Prefixes
*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove this extra line.

Copy link
Contributor

@ajesamson ajesamson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@backstageel backstageel force-pushed the PCHR-3545-Change-Order-For-Individual-Prefix branch from 17ddeb3 to be1ed18 Compare June 19, 2018 15:01
@backstageel backstageel merged commit b054784 into PCHR-3536-Display-Preferences Jun 19, 2018
@backstageel backstageel deleted the PCHR-3545-Change-Order-For-Individual-Prefix branch June 19, 2018 15:02
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.

2 participants