From 4fb24004c2f93707febe629fd7ea0ba09298f0a3 Mon Sep 17 00:00:00 2001
From: danny-cooke-icl <138700430+danny-cooke-icl@users.noreply.github.com>
Date: Mon, 12 Feb 2024 13:46:39 +0000
Subject: [PATCH] update versions

---
 api_app/_version.py                              | 2 +-
 api_app/services/airlock.py                      | 2 +-
 ui/app/package.json                              | 2 +-
 ui/app/src/components/shared/ResourceBody.tsx    | 4 ++--
 ui/app/src/components/shared/airlock/Airlock.tsx | 4 ++--
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/api_app/_version.py b/api_app/_version.py
index 2e93a98791..782e3ece69 100644
--- a/api_app/_version.py
+++ b/api_app/_version.py
@@ -1 +1 @@
-__version__ = "0.18.7"
+__version__ = "0.18.8"
diff --git a/api_app/services/airlock.py b/api_app/services/airlock.py
index 0e5bf1b2d7..19c19a3218 100644
--- a/api_app/services/airlock.py
+++ b/api_app/services/airlock.py
@@ -71,7 +71,7 @@ def validate_user_allowed_to_access_storage_account(user: User, airlock_request:
     allowed_roles = []
 
     if (airlock_request.status == AirlockRequestStatus.InReview):
-        allowed_roles = ["AirlockManager", "WorkspaceOwner"]
+        allowed_roles = ["AirlockManager", "WorkspaceOwner", "ImperialWorkspaceDataEngineer"]
     else:
         allowed_roles = ["WorkspaceResearcher", "WorkspaceOwner", "ImperialWorkspaceDataEngineer"]
 
diff --git a/ui/app/package.json b/ui/app/package.json
index ad32d4d9c9..a152a541c1 100644
--- a/ui/app/package.json
+++ b/ui/app/package.json
@@ -1,6 +1,6 @@
 {
   "name": "tre-ui",
-  "version": "0.5.27",
+  "version": "0.5.28",
   "private": true,
   "dependencies": {
     "@azure/msal-browser": "^2.35.0",
diff --git a/ui/app/src/components/shared/ResourceBody.tsx b/ui/app/src/components/shared/ResourceBody.tsx
index 8d4f5a4dec..a9f01d8591 100644
--- a/ui/app/src/components/shared/ResourceBody.tsx
+++ b/ui/app/src/components/shared/ResourceBody.tsx
@@ -25,14 +25,14 @@ export const ResourceBody: React.FunctionComponent<ResourceBodyProps> = (props:
     [ResourceType.Workspace]: [RoleName.TREAdmin, WorkspaceRoleName.WorkspaceOwner],
     [ResourceType.SharedService]: [RoleName.TREAdmin],
     [ResourceType.WorkspaceService]: [WorkspaceRoleName.WorkspaceOwner],
-    [ResourceType.UserResource]: [WorkspaceRoleName.WorkspaceOwner, WorkspaceRoleName.WorkspaceResearcher, WorkspaceRoleName.ImperialWorkspaceResearcher, WorkspaceRoleName.ImperialWorkspaceOwner]
+    [ResourceType.UserResource]: [WorkspaceRoleName.WorkspaceOwner, WorkspaceRoleName.WorkspaceResearcher, WorkspaceRoleName.ImperialWorkspaceResearcher, WorkspaceRoleName.ImperialWorkspaceOwner, WorkspaceRoleName.ImperialWorkspaceDataEngineer]
   };
 
   const historyRolesByResourceType = {
     [ResourceType.Workspace]: [RoleName.TREAdmin, WorkspaceRoleName.WorkspaceOwner],
     [ResourceType.SharedService]: [RoleName.TREAdmin],
     [ResourceType.WorkspaceService]: [WorkspaceRoleName.WorkspaceOwner],
-    [ResourceType.UserResource]: [WorkspaceRoleName.WorkspaceOwner, WorkspaceRoleName.WorkspaceResearcher, WorkspaceRoleName.ImperialWorkspaceResearcher, WorkspaceRoleName.ImperialWorkspaceOwner]
+    [ResourceType.UserResource]: [WorkspaceRoleName.WorkspaceOwner, WorkspaceRoleName.WorkspaceResearcher, WorkspaceRoleName.ImperialWorkspaceResearcher, WorkspaceRoleName.ImperialWorkspaceOwner, WorkspaceRoleName.ImperialWorkspaceDataEngineer]
   };
 
   const operationsRoles = operationsRolesByResourceType[props.resource.resourceType];
diff --git a/ui/app/src/components/shared/airlock/Airlock.tsx b/ui/app/src/components/shared/airlock/Airlock.tsx
index ea5e245468..9f3194a8e1 100644
--- a/ui/app/src/components/shared/airlock/Airlock.tsx
+++ b/ui/app/src/components/shared/airlock/Airlock.tsx
@@ -347,10 +347,10 @@ export const Airlock: React.FunctionComponent = () => {
       </div>
 
       <Routes>
-        <Route path="new" element={ <SecuredByRole allowedWorkspaceRoles={[WorkspaceRoleName.AirlockManager]} element={
+        <Route path="new" element={ <SecuredByRole allowedWorkspaceRoles={[WorkspaceRoleName.AirlockManager, WorkspaceRoleName.ImperialWorkspaceDataEngineer]} element={
           <AirlockNewRequest onCreateRequest={handleNewRequest} />
         } /> } />
-        <Route path=":requestId" element={ <SecuredByRole allowedWorkspaceRoles={[WorkspaceRoleName.AirlockManager]} element={
+        <Route path=":requestId" element={ <SecuredByRole allowedWorkspaceRoles={[WorkspaceRoleName.AirlockManager, WorkspaceRoleName.ImperialWorkspaceDataEngineer]} element={
           <AirlockViewRequest requests={airlockRequests} onUpdateRequest={getAirlockRequests} />
         } /> } />
       </Routes>