From 5871fa27906d7020fc392ba13504bffa93ae7e49 Mon Sep 17 00:00:00 2001
From: Lukasz Gornicki <lpgornicki@gmail.com>
Date: Tue, 25 Jun 2024 12:56:53 +0200
Subject: [PATCH] fix: remove code needed prior react 18 to fix release of
 webcomponent (#1019)

---
 .../workflows/release-wc-and-playground.yml    | 18 ++++++++++++++++++
 playground/app/page.tsx                        |  1 -
 playground/components/CodeEditorComponent.tsx  |  1 -
 playground/components/SplitWrapper.tsx         |  1 -
 web-component/package.json                     |  4 ++--
 web-component/webpack.config.js                |  2 +-
 6 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/release-wc-and-playground.yml b/.github/workflows/release-wc-and-playground.yml
index 9e7270e3..65d50a21 100644
--- a/.github/workflows/release-wc-and-playground.yml
+++ b/.github/workflows/release-wc-and-playground.yml
@@ -55,6 +55,15 @@ jobs:
           package: ./web-component/package.json
           access: public
           tag: github.event.release.target_commitish
+      - if: failure() # Only, on failure, send a message on the 94_bot-failing-ci slack channel
+        name: Report workflow run status to Slack
+        uses: 8398a7/action-slack@v3
+        with:
+          status: ${{ job.status }}
+          fields: repo,action,workflow
+          text: 'Release of web component for AsyncAPI React failed'
+        env:
+          SLACK_WEBHOOK_URL: ${{ secrets.SLACK_CI_FAIL_NOTIFY }}
 
   playground:
     name: Release Playground
@@ -89,3 +98,12 @@ jobs:
           folder: playground/out
           git-config-name: asyncapi-bot
           git-config-email: info@asyncapi.io
+      - if: failure() # Only, on failure, send a message on the 94_bot-failing-ci slack channel
+        name: Report workflow run status to Slack
+        uses: 8398a7/action-slack@v3
+        with:
+          status: ${{ job.status }}
+          fields: repo,action,workflow
+          text: 'Release of playground for AsyncAPI React failed'
+        env:
+          SLACK_WEBHOOK_URL: ${{ secrets.SLACK_CI_FAIL_NOTIFY }}
\ No newline at end of file
diff --git a/playground/app/page.tsx b/playground/app/page.tsx
index a3a9ba62..66c128f4 100644
--- a/playground/app/page.tsx
+++ b/playground/app/page.tsx
@@ -93,7 +93,6 @@ class Playground extends Component<unknown, State> {
             </Tabs>
           </CodeEditorsWrapper>
           <AsyncApiWrapper>
-            {/* @ts-expect-error remove when library and web-component is upgraded to React v18 */}
             <AsyncApi schema={schema} config={parsedConfig} />
           </AsyncApiWrapper>
         </SplitWrapper>
diff --git a/playground/components/CodeEditorComponent.tsx b/playground/components/CodeEditorComponent.tsx
index d185e441..6d2f90b7 100644
--- a/playground/components/CodeEditorComponent.tsx
+++ b/playground/components/CodeEditorComponent.tsx
@@ -33,7 +33,6 @@ class CodeEditorComponent extends Component<Props, State> {
 
     return (
       <CodeEditorWrapper>
-        {/* @ts-expect-error remove when library and web-component is upgraded to React v18 */}
         <CodeMirror
           value={code}
           basicSetup={{
diff --git a/playground/components/SplitWrapper.tsx b/playground/components/SplitWrapper.tsx
index 2a3a43a7..31fd7fc1 100644
--- a/playground/components/SplitWrapper.tsx
+++ b/playground/components/SplitWrapper.tsx
@@ -8,7 +8,6 @@ interface SplitWrapperProps {
 // react-split should be replaced with a React 18 friendly library or CSS
 const SplitWrapper = (props: SplitWrapperProps) => (
   <>
-    {/* @ts-expect-error upgrade React to v18 */}
     <Split
       style={{
         width: '100%',
diff --git a/web-component/package.json b/web-component/package.json
index f78b756f..38678dbc 100644
--- a/web-component/package.json
+++ b/web-component/package.json
@@ -45,8 +45,8 @@
   },
   "dependencies": {
     "@asyncapi/react-component": "^1.4.10",
-    "react": "^18",
-    "react-dom": "^18",
+    "react": "^18.2.0",
+    "react-dom": "^18.2.0",
     "web-react-components": "^1.4.2"
   },
   "devDependencies": {
diff --git a/web-component/webpack.config.js b/web-component/webpack.config.js
index 4daab1ca..d40e9782 100644
--- a/web-component/webpack.config.js
+++ b/web-component/webpack.config.js
@@ -25,7 +25,7 @@ module.exports = {
       // 2. You might be breaking the Rules of Hooks
       // 3. You might have more than one copy of React in the same app
       // See https://reactjs.org/link/invalid-hook-call for tips about how to debug and fix this problem."
-      react: path.resolve('./node_modules/react'),
+      react: path.resolve('../node_modules/react'),
     },
     fallback: {
       fs: false,