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.

Sign up for GitHub

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

Jump to bottom

Adjust ODM sample to demonstrate automatic mapping #722

Closed
jzheaux opened this issue Jan 19, 2023 · 0 comments
Closed

Adjust ODM sample to demonstrate automatic mapping #722

jzheaux opened this issue Jan 19, 2023 · 0 comments
Assignees
@jzheaux
Labels
in: docs An issue in Documentation or samples type: enhancement
Milestone
3.0.1

Comments

@jzheaux
Copy link
Contributor

jzheaux commented Jan 19, 2023

Confusion like #459 can be alleviated in part by changing the ODM sample to not use @Attribute on fields whose field name matches the LDAP attribute name.

This reinforces the fact that no attribute annotation is needed for the field to be mapped to the entry. And it indirectly implies that @DnAttribute is not creating an entry mapping (since it is already there by default).
@jzheaux jzheaux added type: enhancement in: docs An issue in Documentation or samples labels Jan 19, 2023
@jzheaux jzheaux added this to the 3.0.1 milestone Jan 19, 2023
@jzheaux jzheaux self-assigned this Jan 19, 2023
@jzheaux jzheaux mentioned this issue Jan 19, 2023
Closed
jzheaux added a commit that referenced this issue Jan 19, 2023
@jzheaux
Remove Unneeded Attribute
a9feddd 
Closes gh-722
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jzheaux jzheaux

Labels
in: docs An issue in Documentation or samples type: enhancement
Projects
None yet
Milestone
3.0.1
Development

No branches or pull requests
1 participant
@jzheaux