-
Notifications
You must be signed in to change notification settings - Fork 325
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
Feature/423 add hardcoded attribute mapper #950
Feature/423 add hardcoded attribute mapper #950
Conversation
Changes in the PR are finished and is ready to be merged, we are actively utilizing the newly introduced feature. |
Would love to see this merged and released. I just ran into a configuration change where these mappers are needed. Anything I can do to assist? |
+1, I'm also interested in this feature. |
+1, I'm also interested in this feature. |
+1 |
@angeloxx Are you still interested in landing this? There are some failures because of the move to the new organization and also you need to sign off your commit (see https://git-scm.com/docs/git-commit#Documentation/git-commit.txt--s). |
Signed-off-by: angeloxx <[email protected]>
Signed-off-by: angeloxx <[email protected]>
Signed-off-by: angeloxx <[email protected]>
Signed-off-by: angeloxx <[email protected]>
39e6489
to
7b6ea74
Compare
Signed-off-by: angeloxx <[email protected]>
All checks passed, I updated the PR with the new org and signed off all the commits. Thanks for your time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks a lot!
This PR will include the hardcoded-attribute-mapper (not the hardcoded-ldap-attribute-mapper, still requested in #423)