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

Cannot find type definition file for 'redux' #990

Closed
JesseVarGar opened this issue Aug 13, 2023 · 4 comments · Fixed by #919
Closed

Cannot find type definition file for 'redux' #990

JesseVarGar opened this issue Aug 13, 2023 · 4 comments · Fixed by #919
Assignees

Comments

@JesseVarGar
Copy link
Contributor

Describe the bug

I came across this bug by chance, not because it was affecting anything. It doesn't appear to do anything except highlight line 1 in tsconfig.json. This is the description that VSC gives me for the red highlight on line 1:

Cannot find type definition file for 'redux'.
The file is in the program because:
Entry point for implicit type library 'redux'

To Reproduce

  1. Go to tsconfig.json file
  2. See error on line 1

Expected behavior

No change in behavior from what I can tell.

Machine info (please complete the following information)

  • OS: [Windows 11 Home]
  • Browser & Version [Firefox 116.0.2 (64-bit)]
  • OED Version [0.8.0]
@spearec
Copy link
Contributor

spearec commented Aug 13, 2023

I've noticed this as well. It seems the issue is caused by including src/client/app/types/redux in tsconfig's typeRoots, when it's unclear if we need to do so.
Removing the typeRoots from tsconfig entirely fixes the issue and seems to cause no other problems, but I don't know if that's a valid fix.

@huss
Copy link
Member

huss commented Aug 15, 2023

Hi @JesseVarGar. Thanks for reporting this. Would you like to put in a PR to fix this? If not, someone else can do it.

@JesseVarGar
Copy link
Contributor Author

I don't have much time right now, but I can try to fix it sometime next week if no one gets to it before me.

@huss
Copy link
Member

huss commented Aug 25, 2023

The same error showed up while working on PR #919. I applied the proposed fix and it seems to work fine and will be part of that pull request.

@huss huss self-assigned this Aug 25, 2023
huss added a commit to huss/OED that referenced this issue Aug 25, 2023
@huss huss closed this as completed in #919 Aug 30, 2023
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

Successfully merging a pull request may close this issue.

3 participants