-
Notifications
You must be signed in to change notification settings - Fork 22
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
[Fli iam#1884] Execution monitoring as parent of processing #1946
Conversation
youennmerel
commented
Nov 14, 2023
•
edited
Loading
edited
- Separate Execution monitoring from dataset processing in MS datasets database
- Add a parent_id field to dataset_processing table
- If present, display a link to parent processing in processing detail view
- Disable "Run all" button once submitted (see Disable VIP "Run all" button once clicked #1807)
- Refactor ProcessingService, now ExecutionDataService (see re-write ProcessingService #1898)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost nothing, works fine after changing the mail.
@@ -14,6 +14,7 @@ | |||
|
|||
package org.shanoir.ng.processing.model; | |||
|
|||
import com.fasterxml.jackson.annotation.JsonIgnore; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Useless import
<div *ngIf="!descriptionLoading"> | ||
<h3>Pipeline description : </h3> | ||
<hr> | ||
<p class="warning"><i class="fas fa-exclamation-triangle"></i>Please select a pipeline to see it's description!</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
its
Hi @jcomedouteau, Thx for the review, i tooks your comments into account. |
@jcomedouteau |