From 4414c26a39c37cbcfd956d4067e9b685db449fca Mon Sep 17 00:00:00 2001 From: Jordan Eldredge Date: Thu, 10 Mar 2022 23:55:06 -0500 Subject: [PATCH] Add notion of DELAYED upload processing --- .../skin-database/api/graphql/resolvers/RootResolver.ts | 4 ++++ packages/skin-database/api/graphql/schema.graphql | 6 ++++++ 2 files changed, 10 insertions(+) diff --git a/packages/skin-database/api/graphql/resolvers/RootResolver.ts b/packages/skin-database/api/graphql/resolvers/RootResolver.ts index ee771f10..b533fa25 100644 --- a/packages/skin-database/api/graphql/resolvers/RootResolver.ts +++ b/packages/skin-database/api/graphql/resolvers/RootResolver.ts @@ -112,6 +112,10 @@ class RootResolver extends MutationResolver { // TODO: Could we avoid fetching the skin if it's not read? const skinModel = await SkinModel.fromMd5(ctx, skin_md5); const skin = skinModel == null ? null : new SkinResolver(skinModel); + // Most of the time when a skin fails to process, it's due to some infa + // issue on our side, and we can recover. For now, we'll always tell the user + // That processing is just delayed. + status = status === "ERRORED" ? "DELAYED" : status; return { id, skin, status, upload_md5: skin_md5 }; }) ); diff --git a/packages/skin-database/api/graphql/schema.graphql b/packages/skin-database/api/graphql/schema.graphql index 18f14ac7..28de2cbb 100644 --- a/packages/skin-database/api/graphql/schema.graphql +++ b/packages/skin-database/api/graphql/schema.graphql @@ -318,6 +318,12 @@ enum SkinUploadStatus { the skin will be retried at a later time. """ ERRORED + + """ + An error occured processing the skin, but it was the fault of the server. It + will be processed at a later date. + """ + DELAYED """ The skin has been successfully added to the Museum.