Project

General

Profile

Actions

Task #1157

closed

Long running task detail and clearInterval

Added by Ondřej Kopr over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ondřej Kopr
Category:
Frontend
Target version:
Start date:
06/27/2018
Due date:
% Done:

100%

Estimated time:

Description

Methdo componentWillUnmount in component LongRunningTask clreaing interval but the method isn't called when window is refresh or in go back history behavior.

Please try find solution and repair this bug.


Related issues

Related to CzechIdM - Task #1158: Long running task: use form definition for configurationClosedRadek Tomiška06/28/2018

Actions
Actions #1

Updated by Ondřej Kopr over 4 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Ondřej Kopr over 4 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Ondřej Kopr to Radek Tomiška
  • Priority changed from High to Normal
  • % Done changed from 0 to 70

I found the main problem: I clear interval using clearInterval with undefined and state value, this beahvior brokes clearing interval. I add method for safely clear interval and value is stored in class not state. Now works clearing correctly.

I check these scenarios:
  • go back in history using browser button,
  • replace (paste) url in browser.

Please Radek could you check this behavior.

Actions #4

Updated by Radek Tomiška over 4 years ago

  • Status changed from Needs feedback to Closed
  • Assignee changed from Radek Tomiška to Ondřej Kopr
  • % Done changed from 70 to 100

It works, i was not able to broke it :) Thx!

Note: I removed parameters from bulk operation modal: https://github.com/bcvsolutions/CzechIdMng/commit/0a3e5884fb79546ad6a0b7e5b64391a9db27d0fe. Properties are available on long running task detail. We can show task properties after #1158 will be implemented.

Actions #5

Updated by Radek Tomiška over 4 years ago

  • Related to Task #1158: Long running task: use form definition for configuration added
Actions

Also available in: Atom PDF