fix(widgets): remove non-null assertion in widget edit modal
Satisfies oxlint no-non-null-assertion by narrowing appId before render.
This commit is contained in:
1 file changed
+3
-2
@@ -83,7 +83,8 @@ export const WidgetEditModal = createModal<ModalProps<WidgetKind>>(({ actions, i
|
|||||||
const { openModal } = useModalAction(WidgetAdvancedOptionsModal);
|
const { openModal } = useModalAction(WidgetAdvancedOptionsModal);
|
||||||
|
|
||||||
const canModifyApps = session?.user.permissions.includes("app-modify-all") ?? false;
|
const canModifyApps = session?.user.permissions.includes("app-modify-all") ?? false;
|
||||||
const showAppTab = innerProps.kind === "app" && canModifyApps && Boolean(innerProps.appId);
|
const appId = innerProps.appId;
|
||||||
|
const showAppTab = innerProps.kind === "app" && canModifyApps && Boolean(appId);
|
||||||
|
|
||||||
const handleSubmit = form.onSubmit((values) => {
|
const handleSubmit = form.onSubmit((values) => {
|
||||||
innerProps.onSuccessfulEdit({
|
innerProps.onSuccessfulEdit({
|
||||||
@@ -174,7 +175,7 @@ export const WidgetEditModal = createModal<ModalProps<WidgetKind>>(({ actions, i
|
|||||||
{widgetFormContent}
|
{widgetFormContent}
|
||||||
</Tabs.Panel>
|
</Tabs.Panel>
|
||||||
<Tabs.Panel value="app" pt="md">
|
<Tabs.Panel value="app" pt="md">
|
||||||
<EmbeddedAppEditForm appId={innerProps.appId!} handleRef={appEditRef} />
|
{appId && <EmbeddedAppEditForm appId={appId} handleRef={appEditRef} />}
|
||||||
</Tabs.Panel>
|
</Tabs.Panel>
|
||||||
</Tabs>
|
</Tabs>
|
||||||
<Group justify="end">
|
<Group justify="end">
|
||||||
|
|||||||
Reference in new issue
Block a user