Closed phsmoura closed 1 year ago
Ah, I had prepared #142 for this. I'll compare and try to get the best from both PRs :-)
I merged #142 because it had LDAP update support. It sets the field as multi-valued, I think it's safer to set it as multi-valued in LDAP even if Noggin and FASJSON only support one value for now, because we may want to change that in the future and it's harder to change things in LDAP.
Related: noggin issue 1155 noggin PR 1216