Configure TypeScript / Refactor

landenf / Settlers-of-Catan

Multiplayer Online version of Settlers of Catan

https://catan-4dea2.web.app

2 stars 0 forks source link

Configure TypeScript / Refactor #37

Closed SirFatredVIII closed 3 months ago

SirFatredVIII commented 4 months ago

Added TypeScript to the dependency list
Refactored all components into relevant .TSX filetype
Switched static .JS files over to .TS files

landenf commented 4 months ago

Do we need a typescript config file? https://www.typescriptlang.org/docs/handbook/tsconfig-json.html

SirFatredVIII commented 4 months ago

sorry about the closing / open -- misread the wrong button whoops. recent commit added tsconfig files to the backend and frontend so we should be good now

SirFatredVIII commented 4 months ago

Hesitatnt approval. While running application got 10+ errors. I believe the current syntax would need to be changed on an individaul file level to switch to tsx

Yeah just checked this out -- way worse than I thought it was going to be. Will fix this before our meeting on Wednesday.

SirFatredVIII commented 4 months ago

Newest commit should fix everything nice and well. It sort of bleeds into #36 but I thought these were necessary changes. #36 will be handled further in depth once this PR is merged.