fix: show request access on forbidden only (#1081)

* fix: show request access on forbidden only

* refactor: fix type

Author: pyphilia

src/itemLogin/ItemLoginWrapper.stories.tsx

@@ -1,6 +1,7 @@
 import type { Meta, StoryObj } from '@storybook/react';
 import { expect } from '@storybook/test';
 import { within } from '@storybook/testing-library';
+import { StatusCodes } from 'http-status-codes';
 import { v4 } from 'uuid';
 
 import {
@@ -32,7 +33,7 @@ const meta = {
   args: {
     signIn: () => {},
     itemId: item.id,
-
+    itemErrorStatusCode: null,
     children: <Card alt='card' name='card' />,
   },
 } satisfies Meta<typeof ItemLoginWrapper>;
@@ -93,6 +94,7 @@ export const RequestAccess = {
   args: {
     currentAccount,
     itemId: v4(),
+    itemErrorStatusCode: StatusCodes.FORBIDDEN,
     requestAccessContent: <div data-testId='request'>Request Access</div>,
   },
   play: async ({ canvasElement }) => {

src/itemLogin/ItemLoginWrapper.tsx

@@ -1,3 +1,5 @@
+import { StatusCodes } from 'http-status-codes';
+
 import { ReactElement, ReactNode } from 'react';
 
 import {
@@ -15,6 +17,7 @@ import ItemLoginScreen, { SignInPropertiesType } from './ItemLoginScreen.js';
 export type ItemLoginAuthorizationProps = {
   signIn: (args: { itemId: string } & SignInPropertiesType) => void;
   itemId: UUID;
+  itemErrorStatusCode: number | null;
   currentAccount?: CurrentAccount | null;
   item?: DiscriminatedItem;
   itemLoginSchemaType?: ItemLoginSchemaType;
@@ -31,6 +34,7 @@ export type ItemLoginAuthorizationProps = {
 const ItemLoginAuthorization = ({
   currentAccount,
   item,
+  itemErrorStatusCode,
   itemLoginSchemaType,
   itemId,
   signIn,
@@ -61,8 +65,13 @@ const ItemLoginAuthorization = ({
         return enrollContent ?? forbiddenContent;
       }
 
-      // user is logged in and item login disabled - request access
-      return requestAccessContent ?? forbiddenContent;
+      // user is logged in and item login disabled
+      // cannot access to item - request access
+      if (itemErrorStatusCode === StatusCodes.FORBIDDEN) {
+        return requestAccessContent ?? forbiddenContent;
+      }
+      // any other error return forbidden message
+      return forbiddenContent;
     } else {
       return forbiddenContent;
     }