Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgrade tanstack query #4068

Merged
merged 6 commits into from
Oct 18, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion web/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,8 @@
"@maji/react-prism": "^1.0.1",
"@monaco-editor/react": "^4.4.5",
"@storybook/addon-storysource": "^6.5.16",
"@tanstack/react-query": "^4.36.1",
"@tanstack/react-query-devtools": "^4.36.1",
"accounting": "^0.4.1",
"apexcharts": "^3.24.0",
"axios": "^1.1.2",
Expand Down Expand Up @@ -158,7 +160,6 @@
"react-helmet": "^6.1.0",
"react-icomoon": "^2.5.4",
"react-modal": "3.11.2",
"react-query": "^3.39.1",
"react-refresh": "^0.14.0",
"react-remarkable": "^1.1.3",
"react-router": "^5.1",
Expand Down
2 changes: 1 addition & 1 deletion web/src/Root.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ import UploadAirgapBundle from "./components/UploadAirgapBundle";
import RestoreCompleted from "./components/RestoreCompleted";
import Access from "./components/identity/Access";
import SnapshotsWrapper from "./components/snapshots/SnapshotsWrapper";
import { QueryClient, QueryClientProvider } from "react-query";
import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
import { InstallWithHelm } from "@features/AddNewApp";
import DownstreamTree from "./components/tree/KotsApplicationTree";
import { Dashboard } from "@features/Dashboard/components/Dashboard";
Expand Down
2 changes: 1 addition & 1 deletion web/src/components/hooks/useDownloadValues.test.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
* @jest-environment jsdom
*/
import React from "react";
import { QueryClient, QueryClientProvider } from "react-query";
import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
import { act, renderHook } from "@testing-library/react-hooks";
import { getValues, useDownloadValues } from "./useDownloadValues";

Expand Down
4 changes: 2 additions & 2 deletions web/src/components/hooks/useIsHelmManaged.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery, UseQueryResult } from "react-query";
import { useQuery, UseQueryResult } from "@tanstack/react-query";

interface IsHelmManagedResponse {
isHelmManaged: boolean;
Expand Down Expand Up @@ -30,8 +30,8 @@ async function fetchIsHelmManaged({

function useIsHelmManaged() {
return useQuery({
queryKey: ["isHelmManaged"],
queryFn: () => fetchIsHelmManaged(),
queryKey: "isHelmManaged",
staleTime: Infinity,
select: (response): IsHelmManaged => response.isHelmManaged || false,
});
Expand Down
2 changes: 1 addition & 1 deletion web/src/components/hooks/useSaveConfig.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useMutation } from "react-query";
import { useMutation } from "@tanstack/react-query";

const putConfig = async ({
_fetch = fetch,
Expand Down
2 changes: 1 addition & 1 deletion web/src/components/hooks/useSaveConfig.test.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
* @jest-environment jsdom
*/
import React from "react";
import { QueryClient, QueryClientProvider } from "react-query";
import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
import { renderHook } from "@testing-library/react-hooks";
import { useSaveConfig, putConfig } from "./useSaveConfig";

Expand Down
4 changes: 0 additions & 4 deletions web/src/components/snapshots/SnapshotStorageDestination.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -1831,7 +1831,6 @@ class SnapshotStorageDestination extends Component<Props, State> {
}
/>
</div>

{this.props.showConfigureSnapshotsModal && (
<ConfigureSnapshots
snapshotSettings={this.props.snapshotSettings}
Expand All @@ -1850,7 +1849,6 @@ class SnapshotStorageDestination extends Component<Props, State> {
isKurlEnabled={isKurlEnabled}
/>
)}

{this.state.showConfigureFileSystemProviderModal && (
<Modal
isOpen={this.state.showConfigureFileSystemProviderModal}
Expand All @@ -1863,7 +1861,6 @@ class SnapshotStorageDestination extends Component<Props, State> {
{this.renderConfigureFileSystemProviderModalContent()}
</Modal>
)}

{this.state.showFileSystemProviderInstructionsModal && (
<Modal
isOpen={this.state.showFileSystemProviderInstructionsModal}
Expand Down Expand Up @@ -1894,7 +1891,6 @@ class SnapshotStorageDestination extends Component<Props, State> {
</div>
</Modal>
)}

{showResetFileSystemWarningModal && (
<Modal
isOpen={showResetFileSystemWarningModal}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";

async function getAdminConsoleUpdateStatus({
apiEndpoint = process.env.API_ENDPOINT,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useMutation } from "react-query";
import { useMutation } from "@tanstack/react-query";

async function postUpdateAdminConsole({
apiEndpoint = process.env.API_ENDPOINT,
Expand Down
4 changes: 2 additions & 2 deletions web/src/features/App/api/getApps.test.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
* @jest-environment jsdom
*/
import React from "react";
import { QueryClient, QueryClientProvider } from "react-query";
import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
import { renderHook } from "@testing-library/react-hooks";
import { getApps, useApps } from "./getApps";

Expand All @@ -21,7 +21,7 @@ describe("getApps", () => {
};
});
it("calls _getApps", async () => {
const getAppsSpy = jest.fn(() => Promise.resolve());
const getAppsSpy = jest.fn(() => Promise.resolve({}));

const { result, waitFor } = renderHook(
() => useApps({ _getApps: getAppsSpy }),
Expand Down
4 changes: 2 additions & 2 deletions web/src/features/App/api/getApps.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// This hook has not been integrated yet.
import React from "react";
import { useQuery, UseQueryResult } from "react-query";
import { useQuery, UseQueryResult } from "@tanstack/react-query";
import { Utilities } from "../../../utilities/utilities";
import { App } from "@types";

Expand Down Expand Up @@ -44,7 +44,7 @@ function useApps({
}> {
const query: UseQueryResult<{
apps: App[] | null;
}> = useQuery("apps", () => _getApps(), {
}> = useQuery(["apps"], () => _getApps(), {
refetchInterval,
});

Expand Down
2 changes: 1 addition & 1 deletion web/src/features/App/api/getLicense.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// This hook has not been integrated yet.
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { useSelectedApp } from "@features/App";
import axios from "axios";
import { AppLicense } from "@types";
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/App/api/postDeployAppVersion.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useMutation } from "react-query";
import { useMutation } from "@tanstack/react-query";
import { useNavigate } from "react-router-dom";

async function postDeployAppVersion({
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/App/api/postRedeployAppVersion.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useMutation } from "react-query";
import { useMutation } from "@tanstack/react-query";

async function postRedeployAppVersion({
apiEndpoint = process.env.API_ENDPOINT,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";

async function getAdminConsoleUpdateStatus({
apiEndpoint = process.env.API_ENDPOINT,
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/AppVersionHistory/api/getVersions.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// This hook has not been integrated yet. It's considered a work in progress
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { Utilities } from "../../../utilities/utilities";
import { useParams } from "react-router-dom";
import { useSelectedApp } from "@features/App";
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useMutation } from "react-query";
import { useMutation } from "@tanstack/react-query";

async function postDownloadAppVersion({
apiEndpoint = process.env.API_ENDPOINT,
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/Dashboard/api/createSnapshot.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useMutation } from "react-query";
import { useMutation } from "@tanstack/react-query";
import { useSelectedApp } from "@features/App";

interface SnapshotResponse {
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/Dashboard/api/getAirgapConfig.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { useSelectedApp } from "@features/App";

export const getAirgapConfig = async (appSlug: string): Promise<number> => {
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/Dashboard/api/getAppDownstream.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { isAwaitingResults } from "@src/utilities/utilities";
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { useSelectedApp } from "@features/App";
import { Downstream } from "@types";

Expand Down
2 changes: 1 addition & 1 deletion web/src/features/Dashboard/api/getNextAppVersion.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { useSelectedApp } from "@features/App";
import axios from "axios";

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { useIsHelmManaged } from "@components/hooks";
import { useSelectedApp } from "@features/App";
import { DashboardResponse } from "@types";
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/Dashboard/api/getSnapshotSettings.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { SnapshotSettings } from "@types";
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";

const getSnapshotSettings = async () => {
const res = await fetch(`${process.env.API_ENDPOINT}/snapshots/settings`, {
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/Dashboard/api/getUpdateDownloadStatus.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { useSelectedApp } from "@features/App";

export interface UpdateStatusResponse {
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/Dashboard/api/getUpdates.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { useSelectedApp } from "@features/App";

interface UpdateResponse {
Expand Down
4 changes: 2 additions & 2 deletions web/src/features/Gitops/hooks/useApps.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { Utilities } from "../../../utilities/utilities";

// TODO: replace with fetatures/App/api
// TODO: replace with features/App/api
async function getApps({
apiEndpoint = process.env.API_ENDPOINT,
_fetch = fetch,
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/Gitops/hooks/useGitops.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { Utilities } from "../../../utilities/utilities";

async function getGitops({
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";
import { PreflightCheck, PreflightResponse } from "../types";
import { useState } from "react";

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useMutation, useQueryClient } from "react-query";
import { useMutation, useQueryClient } from "@tanstack/react-query";

async function postIgnorePermissionErrors({
apiEndpoint = process.env.API_ENDPOINT,
Expand Down
2 changes: 1 addition & 1 deletion web/src/features/PreflightChecks/api/postPreflightRun.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useMutation, useQueryClient } from "react-query";
import { useMutation, useQueryClient } from "@tanstack/react-query";

async function postPreflightRun({
apiEndpoint = process.env.API_ENDPOINT,
Expand Down
4 changes: 2 additions & 2 deletions web/src/stores/getMetadata.jsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { useQuery } from "react-query";
import { useQuery } from "@tanstack/react-query";

async function getMetadata({
apiEndpoint = process.env.API_ENDPOINT,
Expand All @@ -19,7 +19,7 @@ async function getMetadata({
}

function useMetadata({ _getMetadata = getMetadata } = {}) {
return useQuery("metadata", () => _getMetadata(), {
return useQuery(["metadata"], () => _getMetadata(), {
staleTime: Infinity,
});
}
Expand Down
1 change: 0 additions & 1 deletion web/webpack.config.dev.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ const ReactRefreshWebpackPlugin = require("@pmmmwh/react-refresh-webpack-plugin"
const ESLintPlugin = require("eslint-webpack-plugin");
const webpack = require("webpack");
const path = require("path");
const srcPath = path.join(__dirname, "src");

module.exports = {
mode: "development",
Expand Down
Loading
Loading