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

Master plan for "username" / "firstname", "lastname" issues #307

Closed
arunz6161 opened this issue Dec 10, 2019 · 2 comments · Fixed by #359
Closed

Master plan for "username" / "firstname", "lastname" issues #307

arunz6161 opened this issue Dec 10, 2019 · 2 comments · Fixed by #359
Assignees
Labels
bug Something isn't working

Comments

@arunz6161
Copy link
Collaborator

No description provided.

@cnovak cnovak added the documentation Documentation related label Mar 3, 2020
@cnovak cnovak changed the title Master doc for "username" / "firstname", "lastname" issues Master plan for "username" / "firstname", "lastname" issues Mar 11, 2020
@cnovak
Copy link
Collaborator

cnovak commented Mar 11, 2020

Overview

There are two "First Name", "Last Name" issues in apigee-edge:

Along with PRs that are still open around these tickets:

Problem Revisited

The issues above are trying to solve the problem that the Apigee Edge module changes the default display to show "Firstname" and "Lastname" instead of username where username is used. This shows up in the logs, the user profile, and list of users.

Before installing Apigee Edge module, the username is properly displayed in list of people and on user profile:
Screen Shot 2020-03-11 at 1 19 55 PM
Screen Shot 2020-03-11 at 1 20 02 PM

After installing Apigee Edge, those same pages look like this:
Screen Shot 2020-03-11 at 1 22 47 PM
Screen Shot 2020-03-11 at 1 22 56 PM

This causes the following problems:

  1. username not displayed in user field, hard to connect to REST API
    : apigee-devportal-kickstart-drupal #270
  2. Username becomes the string "Firstname Lastname" for all users created before Apigee Edge module is installed, as seen above
  3. Log files show "Firstname Lastname" for users in log messages
  4. When installing, the first user is always installed before Apigee Edge module, which means that the user will display as "Firstname Lastname" everywhere which looks odd.

Solution

Approve apigee/apigee-devportal-kickstart-drupal#274 so that Kickstart shows the username again on the profile page.

Create a PR to remove apigee_edge_user_format_name_alter so that Apigee Edge no longer changes username field to display "Firstname Lastname", as @arlina-espinoza pointed out in apigee-edge-drupal #281 that this function is

presentation logic that is not related to this module and is just making things confusing.

We can add in release notes to explain how users can change back that functionality in a custom module if they choose to keep it.

The issue with the "Firstname Lastname" showing up for users will be handled automatically because we are showing the username again for username fields, instead of the Apigee introduced Firstname and Lastname fields.

@arlina-espinoza
Copy link
Contributor

All looks good to me. Added a comment for the release notes on PR#359: #359 (review)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants