fix(artifacts): Persist default artifact account in ExpectedArtifactM…

…odal (spinnaker#6783)
armory-io · Mar 29, 2019 · ce12338 · ce12338
1 parent 3519f6f
commit ce12338
Showing 1 changed file with 15 additions and 0 deletions.
diff --git a/app/scripts/modules/core/src/artifact/react/ArtifactEditor.tsx b/app/scripts/modules/core/src/artifact/react/ArtifactEditor.tsx
@@ -31,6 +31,21 @@ export class ArtifactEditor extends React.Component<IArtifactEditorProps> {
     this.props.onArtifactEdit(artifact);
   };
 
+  private defaultArtifactAccountIfNecessary(): void {
+    const { artifact, artifactAccounts } = this.props;
+    if (!artifact.artifactAccount && artifactAccounts.length > 0) {
+      this.onArtifactAccountChanged(artifactAccounts[0]);
+    }
+  }
+
+  public componentDidMount(): void {
+    this.defaultArtifactAccountIfNecessary();
+  }
+
+  public componentDidUpdate(): void {
+    this.defaultArtifactAccountIfNecessary();
+  }
+
   public render(): React.ReactNode {
     const { pipeline, artifact, artifactAccounts, onArtifactEdit, isDefault } = this.props;
     const artifactAccount = artifactAccounts.find(acc => acc.name === artifact.artifactAccount) || artifactAccounts[0];