Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Request] Added username and emailAddress to extensions #809

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kelvinspencer
Copy link

@kelvinspencer kelvinspencer commented Feb 22, 2022

Description

  • This PR is in reference to a few requests I noticed under issues that were closed, as well as a similar need for a project I am working on. The need is to be able to include an email address in addition to a username to the actor. Unfortunately since we cannot add to the actor schema, I noticed guidance in the comments that the requester should consider adding additional profile attributes to the extensions schema. Reference: Adding custom user profile fields #87 (comment).

  • Please comment if there is an alternative option for including both username and email in the actor schema instead, without modifying the plugin to vastly.

Considerations

  • Leverage Groups in the Actor Schema (Require larger modification of plugin to accomplish than enhancing extensions)
  • Add to Extensions schema (Selected)
  • Develop new schema (Large level of effort)
  • Add to Actor schema (Can't due to restrictions)

Related Issues

PR Type

  • Enhancement

Tests

  • Not yet completed, waiting until confirmed solution is logical and sound by maintainers.
  • Manually tested and confirmed working in the environment I'm actively supporting.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant