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

Added Ability to Import Resources from LDAP (Simple) Currently using #1175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bghost4
Copy link

@bghost4 bghost4 commented Sep 16, 2015

active Directory as a Test Case

Created Dialog to Import from LDAP with Server, BaseDN, Filter,
Username, and Password

Current Support, Imports Resources with Display Name Field, Omits null
results, does not set, Roles

TODO
Import Mail, and Phone Numbers if Available from LDAP.
Bring up a List of people to import in a JTable with a Checkbox Column,
ask Developers best/Standard way to do this.
Support Importing Active Directory/Ldap fields into Custom Fields.
Maybe Add some kind of Mapping to facilitate this.

active Directory as a Test Case

Created Dialog to Import from LDAP with Server, BaseDN, Filter,
Username, and Password

Current Support, Imports Resources with Display Name Field, Omits null
results, does not set, Roles

TODO
	Import Mail, and Phone Numbers if Available from LDAP.
	Bring up a List of people to import in a JTable with a Checkbox Column,
ask Developers best/Standard way to do this. 
	Support Importing Active Directory/Ldap fields into Custom Fields.
Maybe Add some kind of Mapping to facilitate this.
@bghost4
Copy link
Author

bghost4 commented Sep 16, 2015

I couldn't find anything on how to make this a plugin, which is probably the direction I would want to go with this. This is pretty rough around the edges at the moment, I was hoping that this pull request could at least begin some conversation of adding LDAP/AD import for people in domain environments to import and employee list from the Directory.

@dbarashev
Copy link
Contributor

Hey,
thanks for the patch.

May I ask you to format the code using 2 spaces instead of tabs? If you use Eclipse, you can use our formatter settings: https://github.com/bardsoftware/ganttproject/blob/master/ganttproject-builder/formatter.xml

Also, please add GPLv3 license notice to the beginning of the new files and add your name in @author tag in class javadoc comments.

Any instructions how to test this on Linux?

@bghost4
Copy link
Author

bghost4 commented Sep 18, 2015

I work on Linux anyway, and this should be platform independent as far as I know, You would need access to an Active Directory server (Domain) server for this to work. Ideally a Generic LDAP server should work as well. I am working on cleaning up some rough edges and adding an import list to select just the people you want from active directory so It doesn't dump all users. I will update the formatting per your request and Add GPL headers. Coming in my next commit.

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

Successfully merging this pull request may close these issues.

2 participants