From 3aadb65842c4beb57e9ee417e5eca892178c5707 Mon Sep 17 00:00:00 2001 From: noah Date: Sun, 9 Jan 2022 10:53:46 +0900 Subject: [PATCH 1/2] Fix the short ref bug --- ui/src/libs/index.ts | 10 ++++++++++ ui/src/models/index.ts | 2 ++ ui/src/redux/main.ts | 15 ++++++++------- 3 files changed, 20 insertions(+), 7 deletions(-) create mode 100644 ui/src/libs/index.ts diff --git a/ui/src/libs/index.ts b/ui/src/libs/index.ts new file mode 100644 index 00000000..41b5ef97 --- /dev/null +++ b/ui/src/libs/index.ts @@ -0,0 +1,10 @@ +import { Deployment, DeploymentType } from "../models" + +/** + * The function returns the short-formatted ref string. + * @param deployment + * @returns + */ +export const getShortRef = (deployment: Deployment): string => { + return deployment.type === DeploymentType.Commit? deployment.ref.substring(0, 7) : deployment.ref +} \ No newline at end of file diff --git a/ui/src/models/index.ts b/ui/src/models/index.ts index 0de57e78..ae81c1f5 100644 --- a/ui/src/models/index.ts +++ b/ui/src/models/index.ts @@ -18,6 +18,7 @@ import { HttpRequestError, HttpInternalServerError, HttpUnauthorizedError, + HttpPaymentRequiredError, HttpForbiddenError, HttpNotFoundError, HttpConflictError, @@ -50,6 +51,7 @@ export { HttpRequestError, HttpInternalServerError, HttpUnauthorizedError, + HttpPaymentRequiredError, HttpForbiddenError, HttpNotFoundError, HttpConflictError, diff --git a/ui/src/redux/main.ts b/ui/src/redux/main.ts index 464bdcce..f716b20d 100644 --- a/ui/src/redux/main.ts +++ b/ui/src/redux/main.ts @@ -11,6 +11,7 @@ import { EventTypeEnum, HttpInternalServerError, HttpUnauthorizedError, + HttpPaymentRequiredError, License, } from "../models" import { @@ -19,7 +20,7 @@ import { searchReviews as _searchReviews, getLicense } from "../apis" -import { HttpPaymentRequiredError } from "../models/errors" +import { getShortRef } from "../libs" interface MainState { available: boolean @@ -143,26 +144,26 @@ export const notifyDeploymentEvent = createAsyncThunk { const { user } = getState().main - if (event.kind !== EventKindEnum.Deployment) { + if (!(event.kind === EventKindEnum.Deployment && event.deployment)) { return } - if (event.deployment?.deployer?.id !== user?.id) { + if (event.deployment.deployer?.id !== user?.id) { return } if (event.type === EventTypeEnum.Created) { - notify(`New Deployment #${event.deployment?.number}`, { + notify(`New Deployment #${event.deployment.number}`, { icon: "/logo192.png", - body: `Start to deploy ${event.deployment?.ref.substring(0, 7)} to the ${event.deployment?.env} environment of ${event.deployment?.repo?.namespace}/${event.deployment?.repo?.name}.`, + body: `Start to deploy ${getShortRef(event.deployment)} to the ${event.deployment.env} environment of ${event.deployment.repo?.namespace}/${event.deployment.repo?.name}.`, tag: String(event.id), }) return } - notify(`Deployment Updated #${event.deployment?.number}`, { + notify(`Deployment Updated #${event.deployment.number}`, { icon: "/logo192.png", - body: `The deployment ${event.deployment?.number} of ${event.deployment?.repo?.namespace}/${event.deployment?.repo?.name} is updated ${event.deployment?.status}.`, + body: `The deployment ${event.deployment.number} of ${event.deployment.repo?.namespace}/${event.deployment.repo?.name} is updated ${event.deployment.status}.`, tag: String(event.id), }) } From cecd5b7545fb3ceb3ff753038da086e33f1f4233 Mon Sep 17 00:00:00 2001 From: noah Date: Sun, 9 Jan 2022 10:54:45 +0900 Subject: [PATCH 2/2] Add the feed line --- ui/src/libs/index.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ui/src/libs/index.ts b/ui/src/libs/index.ts index 41b5ef97..43fdfd52 100644 --- a/ui/src/libs/index.ts +++ b/ui/src/libs/index.ts @@ -7,4 +7,4 @@ import { Deployment, DeploymentType } from "../models" */ export const getShortRef = (deployment: Deployment): string => { return deployment.type === DeploymentType.Commit? deployment.ref.substring(0, 7) : deployment.ref -} \ No newline at end of file +}