-
Notifications
You must be signed in to change notification settings - Fork 1
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
[WIP] MultiModelEditor; fixes #733; fixes #734 #738
base: development
Are you sure you want to change the base?
[WIP] MultiModelEditor; fixes #733; fixes #734 #738
Conversation
await this.session.Write(transaction.FinalizeTransaction()); | ||
} | ||
} | ||
} |
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.
new line at end of each file please
@@ -222,6 +222,11 @@ public virtual async Task<Result<Iteration>> OpenSession() | |||
return Result.Fail(["The selected iteration and the domain of expertise should not be null"]); | |||
} | |||
|
|||
if (this.sessionService.OpenIterations.Items.Any(x => x.IterationSetup.Iid == this.SelectedIterationSetup.IterationSetupId)) | |||
{ | |||
return Result.Fail(["The selected iteration is already openened"]); |
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.
what happens for the user? if the user should not be able to select an iteration that is already open, lets prevent them from doing so. The feedback message can stay.
{ | ||
await this.OnDragEnd.InvokeAsync((this, node)); | ||
await this.OnCalculateDropIsAllowed.InvokeAsync(this); | ||
Console.WriteLine("DragEnd"); |
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.
can these be implemented using a logging framework ?
@@ -103,6 +104,16 @@ private static List<Application> InitializesApplications() | |||
ComponentType = typeof(ElementDefinitionTable) | |||
}, | |||
|
|||
new TabbedApplication | |||
{ | |||
Name = "Multi Model Editor", |
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.
please change name to Model Editor and this one needs to replace the existing one
@@ -126,6 +126,11 @@ public static class WebAppConstantValues | |||
/// </summary> | |||
public const string ModelEditorPage = "ModelEditor"; |
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.
if this is the old one, i propose we replace it with the new one
/// The shortname of the owning <see cref="DomainOfExpertise" /> | ||
/// </summary> | ||
public string OwnerShortName | ||
{ |
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.
spaces
3495dfc
to
a649e9e
Compare
Prerequisites
Description
MultiModelEditor; fixes #733; fixes #734