refactor: remove legacy client (#4020)

Remove the legacy API client from renku-ui.

Co-authored-by: Lorenzo Cavazzi <43481553+lorenzo-cavazzi@users.noreply.github.com>

Author: leafty

client/src/App.jsx

@@ -120,14 +120,8 @@ export default function App(props) {
     return <Unavailable params={props.params} />;
   }
 
-  const { coreApiVersionedUrlConfig, socket } = props;
   const appContext = {
-    client: props.client,
-    coreApiVersionedUrlConfig,
-    location: props.location,
-    model: props.model,
     params: props.params,
-    webSocket: socket,
   };
 
   return (

client/src/api-client/errors.ts

@@ -16,89 +16,11 @@
  * limitations under the License.
  */
 
-class APIError extends Error {
-  case: string | undefined;
-  response: Response | undefined;
-  error: unknown;
-  errorData: unknown;
-
-  constructor(arg: string | undefined = undefined) {
-    super(arg || "Renku API error");
-  }
-}
-
-const API_ERRORS = {
+export const API_ERRORS = {
   unauthorizedError: "UNAUTHORIZED",
   forbiddenError: "FORBIDDEN",
   notFoundError: "NOT_FOUND",
   internalServerError: "SERVER_ERROR",
   networkError: "NETWORK_PROBLEM",
   authExpired: "AUTH_EXPIRED",
 };
-
-function throwAuthError(response: Response) {
-  const error = new APIError();
-  error.case = API_ERRORS.authExpired;
-  error.response = response;
-  return Promise.reject(error);
-}
-
-function throwErrorWithData(response: Response, data: unknown) {
-  let error;
-  switch (response.status) {
-    case 401:
-      error = new APIError();
-      error.case = API_ERRORS.unauthorizedError;
-      break;
-    case 403:
-      error = new APIError();
-      error.case = API_ERRORS.forbiddenError;
-      break;
-    case 404:
-      error = new APIError();
-      error.case = API_ERRORS.notFoundError;
-      break;
-    case 500:
-      error = new APIError();
-      error.case = API_ERRORS.internalServerError;
-      break;
-    default:
-      error = new APIError();
-  }
-  error.response = response;
-  error.errorData = data;
-  return Promise.reject(error);
-}
-
-function throwAPIErrors(response: Response) {
-  const contentType = response.headers.get("Content-Type");
-  // TODO The default should be to check for type application/json
-  // but I want to make a more minimal change to the code right now.
-  if (contentType === "text/html")
-    return response.text().then((d) => throwErrorWithData(response, d));
-
-  return response.json().then((d) => throwErrorWithData(response, d));
-}
-
-function alertAPIErrors(error: APIError) {
-  switch (error.case) {
-    case API_ERRORS.forbiddenError:
-      throw Error(
-        "You don't have the necessary permission to view this information or perform this action."
-      );
-    case API_ERRORS.notFoundError:
-      throw Error("We could not find the requested resource on the server.");
-    case API_ERRORS.internalServerError:
-      throw Error(
-        "There is a problem with the server - please try again later."
-      );
-    case API_ERRORS.networkError:
-      throw Error(
-        "There seems to be problem with your network connection. Please check and try again."
-      );
-    default:
-    // No alert on default exception
-  }
-}
-
-export { APIError, API_ERRORS, alertAPIErrors, throwAPIErrors, throwAuthError };