Skip to content

Commit 0c9fdb5

Browse files
committed
don't use default value for array prop as svelte handles them in inconsistent way
Signed-off-by: Josh Hootman <[email protected]>
1 parent 2b7a85e commit 0c9fdb5

File tree

3 files changed

+12
-14
lines changed

3 files changed

+12
-14
lines changed

frontend/src/lib/components/SelectedValues.svelte

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<script lang="ts">
22
import type { SelectedValue } from '$lib/services/Libre311/Libre311';
33
4-
export let selectedValues: SelectedValue[] = [];
4+
export let selectedValues: SelectedValue[];
55
</script>
66

77
{#each selectedValues as selectedValue}

frontend/src/lib/components/ServiceRequestDetails.svelte

+5-4
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,11 @@
5858
<p class="text-sm">{serviceRequest.address}</p>
5959
</div>
6060

61-
<div class="mb-1">
62-
<SelectedValues selectedValues={serviceRequest.selected_values} />
63-
</div>
64-
61+
{#if serviceRequest.selected_values}
62+
<div class="mb-1">
63+
<SelectedValues selectedValues={serviceRequest.selected_values} />
64+
</div>
65+
{/if}
6566
{#if serviceRequest.description}
6667
<div class="mb-1">
6768
<strong class="text-base">{messages['serviceRequest']['description']}</strong>

frontend/src/lib/components/ServiceRequestUpdate.svelte

+6-9
Original file line numberDiff line numberDiff line change
@@ -103,10 +103,7 @@
103103
}
104104
105105
async function updateServiceRequest() {
106-
if (agencyNameInput.value !== null || agencyNameInput.value !== undefined) {
107-
let agency: string = new String(agencyNameInput.value).toString();
108-
if (agency) agencyNameInput = optionalCoalesceNameValidator(createInput(agency));
109-
}
106+
agencyNameInput = optionalCoalesceNameValidator(agencyEmailInput);
110107
agencyEmailInput = emailValidator(agencyEmailInput);
111108
serviceNoticeInput = optionalCoalesceStringValidator(serviceNoticeInput);
112109
statusNotesInput = optionalCoalesceStringValidator(statusNotesInput);
@@ -118,8 +115,6 @@
118115
statusNotesInput.type
119116
]);
120117
if (resultSet.has('invalid')) {
121-
// Refresh Inputs
122-
agencyNameInput.value = '';
123118
return;
124119
}
125120
@@ -175,9 +170,11 @@
175170
<p class="text-sm">{serviceRequest.address}</p>
176171
</div>
177172

178-
<div class="mb-1">
179-
<SelectedValues selectedValues={serviceRequest.selected_values} />
180-
</div>
173+
{#if serviceRequest.selected_values}
174+
<div class="mb-1">
175+
<SelectedValues selectedValues={serviceRequest.selected_values} />
176+
</div>
177+
{/if}
181178

182179
{#if serviceRequest.description}
183180
<div class="mb-1">

0 commit comments

Comments
 (0)