Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typescript errors on default install/up #187

Open
mkbt opened this issue Apr 16, 2022 · 3 comments
Open

Typescript errors on default install/up #187

mkbt opened this issue Apr 16, 2022 · 3 comments

Comments

@mkbt
Copy link

mkbt commented Apr 16, 2022

Hi there,

I'm getting a bunch of Typescript errors in the browser console when bringing the docker-compose file up from defaults.

VM111:2 Uncaught ReferenceError: process is not defined
    at Object.4043 (<anonymous>:2:13168)
    at r (<anonymous>:2:306599)
    at Object.8048 (<anonymous>:2:9496)
    at r (<anonymous>:2:306599)
    at Object.8641 (<anonymous>:2:1379)
    at r (<anonymous>:2:306599)
    at <anonymous>:2:315627
    at <anonymous>:2:324225
    at <anonymous>:2:324229
    at HTMLIFrameElement.e.onload (index.js:1:1)
TypeScript error in /app/src/index.tsx(8,4):
'Router' cannot be used as a JSX component.
  Its instance type 'BrowserRouter' is not a valid JSX element.
    The types returned by 'render()' are incompatible between these types.
      Type 'React.ReactNode' is not assignable to type 'import("/node_modules/@types/react-transition-group/node_modules/@types/react/index").ReactNode'.  TS2786

     6 | 
     7 | ReactDOM.render(
  >  8 |   <Router>
       |    ^
     9 |     <App />
    10 |   </Router>,
    11 |   document.getElementById('root')

Is there a dependancy I'm missing?

I tried building from the latest commits via a fork mentioned at #177 but still ran into further issues.

@gegnew
Copy link

gegnew commented Jun 24, 2022

Any update on this? Also curious

@hung96ad
Copy link

hung96ad commented Jul 7, 2022

i same you :(

@aeciorc
Copy link

aeciorc commented Jul 22, 2022

Update typescript to 4.1.0, then add this to your package.json at the root level:

 "resolutions": {
    "@types/react": "^17.0.38"
  },

You're still going to run into a ton of other issues though, from dependency mismatches to missing libraries. Fixing the MUI issues will be a pain. I honestly think it's not worth using the frontend portion of this app at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants