Project

General

Profile

Actions
Copy link

Task #2354

closed

ImportRolesFromCSVExecutor: add support for importing role EAVs and setting multiple system mappings in one LRT

Added by Tomáš Doischer over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Tomáš Doischer
Target version:
2.6.0
Start date:
06/30/2020
Due date:
% Done:

100%

Estimated time:
Owner:

Description

The goal of this ticket is to:

  1. enable setting a value of role EAV when creating/updating a role
  2. enable setting multiple system mappings for one LRT when creating/updating a role, i. e., move the name of the system to the the CSV and make it multivalued

The issue with 2. is that with multiple systems, you might want to set different attributes and attribute values to each. Probably the best way to do this would be to merge system, attribute code and attribute value into one column, separated by a special character. For example:

role;system
testrole;AD+ldapGroups+AD_all

The separator can be then specified in the LRT config.

Actions
Copy link
 #1

Updated by Tomáš Doischer over 4 years ago

  • Status changed from New to Needs feedback
  • % Done changed from 0 to 80

Implemented in branch https://github.com/bcvsolutions/czechidm-extras/tree/tdoischer/2354-improve-importRoles

There are tests for new features. The documentation was updated (https://wiki.czechidm.com/tutorial/adm/roles_-_import_data_from_csv).

@sourek, can you please do a code review?

Actions
Copy link
 #2

Updated by Peter Štrunc over 4 years ago

  • Target version changed from 2.5.0 to 2.6.0

Postponing to 2.6.0

Actions
Copy link
 #3

Updated by Peter Štrunc over 4 years ago

  • Status changed from Needs feedback to Closed
  • % Done changed from 80 to 100

Thanks for this feature. it looks great. It will be merged to develop along with #2421. Closing this ticket.

Actions
Copy link

Also available in: Atom PDF