Skip to content
Snippets Groups Projects
Commit 91f19b94 authored by Sam Kagan's avatar Sam Kagan
Browse files

Merge branch 'fix-secci-planes-checklist' into 'release/2.4.3-rc1'

Fixed bug where deselected planes were included in API call

See merge request !23
parents 73f4ba6f 0eef036b
No related branches found
No related tags found
2 merge requests!25Merge 2.4.3 UI to main,!23Fixed bug where deselected planes were included in API call
......@@ -41,7 +41,7 @@
</h4>
<div class="w-100" *ngFor="let plane of planeKeys" [formGroup]="planesFormGroup">
<input type="checkbox" value="{{plane}}" checked/> <label>{{plane}}</label>
<label><input type="checkbox" value="{{plane}}" formControlName="{{plane}}" checked/> {{plane}}</label>
</div>
</div>
<div *ngIf="canSelectTiles()">
......
......@@ -233,11 +233,11 @@ export class ExecutionDetailComponent implements OnInit, OnDestroy, OnChanges {
// Collect the selected planes and write their names separated by a newline
let planesText = '';
const planes = this.planesFormGroup.value;
Object.keys(planes).forEach(key => {
if (planes[key] === true) {
planesText += key + '\n';
for (const planeKey in planes) {
if (planes[planeKey] === true) {
planesText += planeKey + "\n";
}
});
}
planesText = planesText.trim();
// Write out the planes string if any are selected
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment