From 2dbb06f433964ccac13fd64ef512ed03142ed97b Mon Sep 17 00:00:00 2001
From: Fabian Mastenbroek <mail.fabianm@gmail.com>
Date: Tue, 11 May 2021 16:34:25 +0200
Subject: ui: Move communication to REST API

This change removes the socket.io websocket connection/client in favour
of the OpenDC REST API. The socket.io websocket implementation was
intended to be used for interactive and collaborative datacenter design
and exploration.

However, we do not support this functionality at the moment
(collaborative design and exploration) and having the entire API run
over this websocket connection is fragile and not standard practice.

To improve maintainability, we therefore remove the websocket
implementation in favour of the OpenDC REST API implementation using
the fetch API. If we want to implement collaboration in the future, we
will develop appropriate extensions in conjuction with the existing REST
API. For this, we should look for standard and existing implementation
of this functionality.
---
 opendc-web/opendc-web-ui/src/index.js | 37 ++++++++++++++++-------------------
 1 file changed, 17 insertions(+), 20 deletions(-)

(limited to 'opendc-web/opendc-web-ui/src/index.js')

diff --git a/opendc-web/opendc-web-ui/src/index.js b/opendc-web/opendc-web-ui/src/index.js
index d40d17a2..fdfec24b 100644
--- a/opendc-web/opendc-web-ui/src/index.js
+++ b/opendc-web/opendc-web-ui/src/index.js
@@ -3,30 +3,27 @@ import ReactDOM from 'react-dom'
 import * as Sentry from '@sentry/react'
 import { Integrations } from '@sentry/tracing'
 import { Provider } from 'react-redux'
-import { setupSocketConnection } from './api/socket'
 import './index.scss'
 import Routes from './routes'
 import config from './config'
 import configureStore from './store/configure-store'
 
-setupSocketConnection(() => {
-    const store = configureStore()
+const store = configureStore()
 
-    // Initialize Sentry if the user has configured a DSN
-    const dsn = config['SENTRY_DSN']
-    if (dsn) {
-        Sentry.init({
-            environment: process.env.NODE_ENV,
-            dsn: dsn,
-            integrations: [new Integrations.BrowserTracing()],
-            tracesSampleRate: 0.1,
-        })
-    }
+// Initialize Sentry if the user has configured a DSN
+const dsn = config['SENTRY_DSN']
+if (dsn) {
+    Sentry.init({
+        environment: process.env.NODE_ENV,
+        dsn: dsn,
+        integrations: [new Integrations.BrowserTracing()],
+        tracesSampleRate: 0.1,
+    })
+}
 
-    ReactDOM.render(
-        <Provider store={store}>
-            <Routes />
-        </Provider>,
-        document.getElementById('root')
-    )
-})
+ReactDOM.render(
+    <Provider store={store}>
+        <Routes />
+    </Provider>,
+    document.getElementById('root')
+)
-- 
cgit v1.2.3