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

Resolve missing types in TypeScript 5 #1338

Merged
merged 2 commits into from Jun 9, 2023

Conversation

leachtucker
Copy link
Contributor

@leachtucker leachtucker commented Jun 7, 2023

Features and Changes

  • Resolves missing type declarations error in TypeScript 5 projects with moduleResolution set to bundler

Screenshots

The resolved TS Error:
TS Error

@bttf
Copy link
Collaborator

bttf commented Jun 8, 2023

Hi @leachtucker Could you specify which bundler you are using?

@leachtucker
Copy link
Contributor Author

Hey @bttf, we are using Vite. Although, this issue is not specific to Vite -- it occurs for any TS projects utilizing the new bundler option as its moduleResolution

@bttf
Copy link
Collaborator

bttf commented Jun 9, 2023

Hi @leachtucker Thanks for your patience. I was able to repro and the changes look good on my end. Will merge by EOD

@leachtucker
Copy link
Contributor Author

Hi @leachtucker Thanks for your patience. I was able to repro and the changes look good on my end. Will merge by EOD

Hey @bttf. That sounds good! Thank you 🙂

@bttf bttf merged commit 55e694f into growthbook:main Jun 9, 2023
2 of 3 checks passed
@chambo-e
Copy link

Hello @bttf 👋

Would it be possible to release a new version of the packages to include this fix please ?

Thanks :)

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 this pull request may close these issues.

None yet

3 participants