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

CCB: Non-unique Group Ids created on Director groups #73

Open
guymaness2 opened this issue Jun 29, 2022 · 0 comments
Open

CCB: Non-unique Group Ids created on Director groups #73

guymaness2 opened this issue Jun 29, 2022 · 0 comments

Comments

@guymaness2
Copy link

guymaness2 commented Jun 29, 2022

Current Behavior
The current code for CCB data extraction automatically breaks out Department Directors into sub-groups beneath their respective Department, placing all groups within that department beneath the new Director level group rather than the Department level group. The GroupId created for these Director level groups is generated by prepending "9998" to the CCB director id, then converting the result to an int. This results in unique ids for every director.

Issue
When the same Director is assigned to multiple Departments in CCB, the current design for generating the Director level GroupId causes problems as they are only unique to the Director, not to the Director/Department. All of the Director groups exist in the exported package with the appropriate Department group as their parent group as expected. However, since their GroupId is only unique to the Director, the Bulk Import block only imports the first group it comes across with a given Director GroupId. This results in an unpredictable (and somewhat unusable) group hierarchy for any Director assigned to more than one Department, as they will have only one group created in one of their departments with all groups from any departments that have them as a director under that group. All relationship data between department and lowest level group is lost, so post import cleanup is not possible.

Proposed Solution
Update the logic building GroupId for the Director group to be unique for every Director/Department combination rather than just Director.

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

No branches or pull requests

1 participant