fix: address PR review findings
- Revert undici to v7.27.1 (v8 blocked by #5467 qBittorrent regression) - Broaden cache invalidation to utils.integration.invalidate() (covers all integration queries, not just integration.all) - Add invalidation to onboarding integration create path - Harden renovate test to assert allowBuilds key exists - Remove deprecated Vitest 4 poolOptions config - Fix CSS typo in docs menu transition (0.s → 0.5s) - Update renovate.json5 comment to reference allowBuilds
This commit is contained in:
9 files changed
+181
-88
No files matched your search
@@ -93,7 +93,7 @@
|
||||
}
|
||||
|
||||
.menu__list {
|
||||
transition: all 0.s ease-in-out !important;
|
||||
transition: all 0.5s ease-in-out !important;
|
||||
}
|
||||
|
||||
.announcementBar_node_modules-\@docusaurus-theme-classic-lib-theme-AnnouncementBar-styles-module {
|
||||
|
||||
@@ -44,7 +44,7 @@ export const DeleteIntegrationActionButton = ({ count, integration }: DeleteInte
|
||||
router.replace("/manage/integrations");
|
||||
}
|
||||
void revalidatePathActionAsync("/manage/integrations");
|
||||
void utils.integration.all.invalidate();
|
||||
void utils.integration.invalidate();
|
||||
},
|
||||
onError: () => {
|
||||
showErrorNotification({
|
||||
|
||||
+1
-1
@@ -103,7 +103,7 @@ export const EditIntegrationForm = ({ integration }: EditIntegrationForm) => {
|
||||
title: t("integration.page.edit.notification.success.title"),
|
||||
message: t("integration.page.edit.notification.success.message"),
|
||||
});
|
||||
void utils.integration.all.invalidate();
|
||||
void utils.integration.invalidate();
|
||||
void revalidatePathActionAsync("/manage/integrations").then(() => router.push("/manage/integrations"));
|
||||
},
|
||||
onError: () => {
|
||||
|
||||
@@ -101,11 +101,15 @@ export const NewIntegrationForm = ({
|
||||
const { mutateAsync: createIntegrationAsync, isPending: isCreatePending } = clientApi.integration.create.useMutation({
|
||||
async onSuccess() {
|
||||
await revalidatePathActionAsync("/manage/integrations");
|
||||
await utils.integration.all.invalidate();
|
||||
await utils.integration.invalidate();
|
||||
},
|
||||
});
|
||||
const { mutateAsync: createOnboardingIntegrationAsync, isPending: isOnboardingCreatePending } =
|
||||
clientApi.onboard.createIntegration.useMutation();
|
||||
clientApi.onboard.createIntegration.useMutation({
|
||||
async onSuccess() {
|
||||
await utils.integration.invalidate();
|
||||
},
|
||||
});
|
||||
const isPending = isCreatePending || isOnboardingCreatePending;
|
||||
const [error, setError] = useState<null | AnyMappedTestConnectionError>(null);
|
||||
|
||||
|
||||
Reference in new issue
Block a user