Placeholder for Change requests

Author: oharsta

client/src/api/index.js

@@ -185,7 +185,6 @@ export function privacy() {
     return fetchJson("/api/v1/manage/privacy", {}, {}, false);
 }
 
-
 //Connections
 export function newConnection(connection) {
     return postPutJson("/api/v1/connections", connection, "POST");
@@ -211,6 +210,11 @@ export function requestConnectionProductionStatus(connectionId) {
     return postPutJson(`/api/v1/connections/request-production-status/${connectionId}`, {}, "PUT");
 }
 
+export function getChangeRequests(connection) {
+    return fetchJson(`/api/v1/connections/change-requests/${connection.id}`, {}, {}, false);
+}
+
+
 //OrganizationMemberships
 export function changeOrganizationMembershipById(organizationMembership, authority) {
     const body = {id: organizationMembership.id, authority: authority}

client/src/connection/ChangeRequests.jsx

@@ -0,0 +1,55 @@
+import "./ChangeRequests.scss";
+import React, {useEffect, useState} from "react";
+import I18n from "../locale/I18n";
+import {useAppStore} from "../stores/AppStore.js";
+import {getChangeRequests} from "../api/index.js";
+import {Button, Loader} from "@surfnet/sds";
+import {ENVIRONMENTS} from "../utils/Manage.js";
+
+export const ChangeRequests = ({
+                                   application
+                               }) => {
+
+    const {setFlash} = useAppStore(state => state);
+    const [changeRequests, setChangeRequests] = useState(true);
+    const [loading, setLoading] = useState(true);
+
+    useEffect(() => {
+        const promises = application.connections
+            .filter(connection => connection.environment === ENVIRONMENTS.PROD && connection.state === "prodaccepted")
+            .map(connection => getChangeRequests(connection));
+        Promise.all(promises)
+            .then(res => {
+                setChangeRequests(res);
+                setLoading(false);
+            })
+
+    }, [application]);
+
+
+    if (loading) {
+        return <Loader/>
+    }
+
+    return (
+        <div className="change-requests-container">
+            <div className="change-requests-inner">
+                <div className="app-header">
+                    <h2>{I18n.t("changeRequests.title")}</h2>
+                </div>
+                <div className="change-requests-information card">
+                    {JSON.stringify(changeRequests)}
+                </div>
+                <div className={`actions orphan`}>
+                    {!application.signedContract &&
+                        <>
+                            <Button txt={I18n.t("connection.contractSection.sign")}
+                                    disabled={!maySignContract}
+                                    onClick={() => submit()}/>
+                        </>
+                    }
+                </div>
+            </div>
+        </div>
+    );
+}

client/src/connection/ChangeRequests.scss

@@ -0,0 +1,38 @@
+@use "../styles/mixins" as *;
+@use "../index";
+@use "../styles/application";
+
+$width-percentage: 75%;
+
+.change-requests-container {
+    background-color: white;
+    padding: 25px 50px;
+
+    .change-requests-inner {
+        max-width: 60%;
+    }
+    .app-header {
+        display: flex;
+        align-items: center;
+        margin-bottom: 25px;
+    }
+    .change-requests-information {
+        max-width: 100%;
+    }
+    .actions {
+        display: flex;
+        margin: 25px 0;
+        align-items: flex-end;
+        justify-content: space-between;
+
+        &.orphan {
+            justify-content: flex-end;
+        }
+
+
+        button svg {
+            width: 16px;
+            height: auto;
+        }
+    }
+}

client/src/connection/Testing.jsx

@@ -1015,6 +1015,9 @@ export const Testing = ({
                 return !informationProfileValid();
             }
             case sections.testIdP: {
+                if (isProduction) {
+                    return false;
+                }
                 return !testIdPValid();
             }
         }
@@ -1034,7 +1037,7 @@ export const Testing = ({
         }
         const proceed = (section === sections.technical && technicalValid()) ||
             (section === sections.informationProfile && informationProfileValid()) ||
-            (section === sections.testIdP && testIdPValid());
+            (section === sections.testIdP && (isProduction || testIdPValid()));
         if (proceed) {
             setLoading(true);
             const promise = connection.id ? updateConnection : newConnection

client/src/locale/en.js

@@ -198,6 +198,7 @@ const en = {
         testing: "Testing",
         prod: "Production",
         application: "App information",
+        change_requests: "Change request",
         contract: "Contract",
         appteam: "Appteam",
         welcome: "Welkom {{user}}. {{name}} is nog niet gekoppeld met SURF Access. Begin met een koppeling aan onze testomgeving.",

client/src/pages/Connection.jsx

@@ -20,8 +20,9 @@ import {Contract} from "../connection/Contract.jsx";
 import {AppTeamManagement} from "../application/AppTeamManagement.jsx";
 import {connectOptions, visibilities} from "../utils/Connection.js";
 import {isEmpty} from "../utils/Utils.js";
+import {ChangeRequests} from "../connection/ChangeRequests.jsx";
 
-const tabNames = ["overview", "testing", "prod", "application", "contract", "appteam"]
+const tabNames = ["overview", "testing", "prod", "application", "change_requests", "contract", "appteam"]
 
 const protocolOptions = Object.values(PROTOCOLS).map(protocol => ({
     value: protocol,
@@ -214,6 +215,10 @@ export const Connection = () => {
                                        arpInfo={arp}
                 />
             }
+            case "change_requests": {
+                return <ChangeRequests application={application}
+                />
+            }
             case "contract": {
                 return <Contract application={application}
                                  setApplication={setApplication}

server/src/test/java/access/manage/ConnectionProviderConverterTest.java

@@ -20,6 +20,7 @@ class ConnectionProviderConverterTest extends AbstractTest {
 
     @Autowired
     private ConnectionProviderConverter connectionProviderConverter;
+
     @Autowired
     private Manage manage;