Project

General

Profile

Actions
Copy link

Feature #3152

closed

Manager (owner) of system

Added by Roman Kučera over 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Roman Kučera
Category:
Technical and other accounts
Target version:
13.0.0
Start date:
06/27/2022
Due date:
% Done:

100%

Estimated time:
8.00 h
Owner:

Description

Junction table.
Defined directly or by role.
Will be used in technical account module

Actions
Copy link
 #1

Updated by Roman Kučera over 2 years ago

  • Status changed from New to In Progress
  • Assignee set to Roman Kučera
Actions
Copy link
 #2

Updated by Roman Kučera over 2 years ago

  • % Done changed from 0 to 70

implementation in https://github.com/bcvsolutions/CzechIdMng/tree/kucerar/3152-system-owner
BE should be implemented, but most REST tests are failing for new controller, I need to look at it.

Actions
Copy link
 #3

Updated by Roman Kučera over 2 years ago

  • % Done changed from 70 to 80

Implemented the rest of BE (event types and processors)
REST tests for this new feature are passing.

Next I will probably write some more integration test.

Actions
Copy link
 #4

Updated by Roman Kučera over 2 years ago

  • Status changed from In Progress to Needs feedback
  • % Done changed from 80 to 90

Evaluators are implemented to.
Added tests.

PR https://github.com/bcvsolutions/CzechIdMng/pull/231

Actions
Copy link
 #5

Updated by Roman Kučera over 2 years ago

  • Assignee changed from Roman Kučera to Tomáš Doischer

@doischert can you make a review please?

Actions
Copy link
 #6

Updated by Tomáš Doischer over 2 years ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Tomáš Doischer to Roman Kučera

I have only very minor notes, see the PR, please.

Actions
Copy link
 #7

Updated by Roman Kučera over 2 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Roman Kučera to Tomáš Doischer

PR is updated.
I also added flyway script.
I tested the flyway and then run some HTTP request to API directly to test it, because there is no GUI for this feature for now.

@doischert you can continue with review, thx.

Actions
Copy link
 #8

Updated by Tomáš Doischer over 2 years ago

  • Sprint set to Sprint 12.3-1 (srp 03 - srp 17)
Actions
Copy link
 #9

Updated by Roman Kučera over 2 years ago

Everything is fixed and PR check is green https://github.com/bcvsolutions/CzechIdMng/pull/231

Actions
Copy link
 #10

Updated by Tomáš Doischer over 2 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Tomáš Doischer to Roman Kučera
  • % Done changed from 90 to 100

LGTM, merged to develop.

Actions
Copy link
 #11

Updated by Tomáš Doischer almost 2 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF