Project

General

Profile

Actions

Task #786

closed

Target system detail - tab with roles

Added by Radek Tomiška over 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Low
Assignee:
Petr Hanák
Category:
Systems
Target version:
Start date:
10/24/2017
Due date:
% Done:

100%

Estimated time:
8.00 h
Owner:

Description

Create new tab on system detail with assigned roles - the same as tab with target systems on role detail. Assing new role to system will be possible.


Files

detail_btn.png (729 Bytes) detail_btn.png Ondřej Kopr, 01/02/2018 11:49 AM
Actions #1

Updated by Petr Hanák over 6 years ago

  • Assignee changed from Radek Tomiška to Petr Hanák
Actions #2

Updated by Petr Hanák over 6 years ago

  • Assignee changed from Petr Hanák to Ondřej Kopr
Actions #3

Updated by Ondřej Kopr over 6 years ago

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

Please for next time change state of task to needs feedback :)

edit: and give me some information about commits/branch where I find this feature + some minimalist information about your new behavior is also good idea.

Actions #4

Updated by Ondřej Kopr over 6 years ago

  • File detail_btn.png detail_btn.png added
  • Status changed from Needs feedback to In Progress
  • Assignee changed from Ondřej Kopr to Petr Hanák
  • % Done changed from 90 to 70

I made first feedback:

  • in FE component SystemRoles is defined method showDetail, this method is necessary remove it,
  • into system role table please add detail button: ,
  • Advanced.Column hasn't props access please remove it,
  • eslint annotation for multi component isn't required more you can remove it,
  • set sort 'dsc' for search parameter isn't working, use desc or asc,
  • in localization you use 'entity.IdentityRole' but this is role try 'entity.Role',
  • in select method you defined role and _showLoading props but these props is never used you can remove it,
  • you are add some localization items to core entity IdentityRole, please remove it, use 'entity.Role',
  • content key is 'acc:content.systemRoles' but probably better is 'acc:content.system.roles', for example like in system connector,
  • please add filter to this table for role.

After you complete this feedback return the task to me, i will continue with review.

Actions #5

Updated by Petr Hanák over 6 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Petr Hanák to Ondřej Kopr
Actions #6

Updated by Ondřej Kopr over 6 years ago

  • Assignee changed from Ondřej Kopr to Radek Tomiška
  • % Done changed from 70 to 90

I found two yours branches, that correspond with this ticket, please for next time give me information in what branch is your new feature:
https://github.com/bcvsolutions/CzechIdMng/tree/786-assigned-roles-tab-in-system-detail
https://github.com/bcvsolutions/CzechIdMng/tree/phanak/786-assigned-roles-tab-in-system-detail

Into your original branch I already merged develop I must do this again.

Second feedback (it's only information's, I already fix all these things):
  • unused import UUID, for next time check eslint,
  • your new filter doesn't work:
    Uncaught TypeError: Cannot read property 'getDefaultSearchParameters' of undefined
        at FilterSelectBox._createSearchParameters (app.js:45021)
        at FilterSelectBox.getOptions (app.js:45047)
        at FilterSelectBox.onOpen (app.js:45571)
        at Object.componentWillUpdate (app.js:24415)
    

    for next time try it, you forgot add manager into your select box,
  • your new filter filtering by roleSystem, but SysRoleSystemFilter doesn't has this property,
  • you still don't remove props role and _showLoading,
  • localization items aren't still removed,

I already removed old branch. Current branch: https://github.com/bcvsolutions/CzechIdMng/tree/phanak/786-assigned-roles-tab-in-system-detail
My commit: https://github.com/bcvsolutions/CzechIdMng/commit/05683632c2a7b802031c18da2c3f89962aa26821
pull request: https://github.com/bcvsolutions/CzechIdMng/pull/12

Please Radek could you made a last review? Thank you.

Actions #7

Updated by Radek Tomiška about 6 years ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Radek Tomiška to Petr Hanák
  • % Done changed from 90 to 50

I did test and review, new tab with roles works basically, we need to add some improvement:
- table is copied from original tab - crate pls new component with table and use her on both places (see IdentityTable and their props - forceSearchParameters, columns ...)
- add button is missing
- detail button leads to role detail - RoleSystem detail should be shown
- remove defining sort in forceSearchParameters - this means table cannot be sorted by user
- use info cards in table for role and system columns. Hide one of this column when role detail tab or system detail tab is used (see prop columns usage)

Actions #8

Updated by Petr Hanák about 6 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Petr Hanák to Radek Tomiška
  • % Done changed from 50 to 80

My adjustments:
- created new component - RoleSystemTable
- this component is used in RoleSystems and SystemRoles tabs
- add button added to RoleSystems tab (I was not sure how to implement it on SystemRoles)
- detail button is repaired and points to RoleSystem detail from both places
- table can be sorted by user
- added select boxes
- added info cards

Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/7027ca44e67f4f1abd46fdf0db85df811e2ea167

Actions #9

Updated by Radek Tomiška almost 6 years ago

I did test and review, new tab works, thx!

I did a refactoring and I added unimplemented operations - add and update attribute mapping - from system menu, commit:
https://github.com/bcvsolutions/CzechIdMng/commit/786d5f9079c52c0abc521555a2419b79b6387aa8

It's merged into develop.

Actions #10

Updated by Radek Tomiška almost 6 years ago

  • Status changed from Needs feedback to Closed
  • Assignee changed from Radek Tomiška to Petr Hanák
  • Target version set to Jade (8.1.0)
  • % Done changed from 80 to 100
Actions

Also available in: Atom PDF