search

aralroca / next-translate-plugin

Next-translate plugin for i18n in Next.js 🌍 - Load page translations and use them in an easy way!
MIT License
30 stars 17 forks source link

fix(utils): Load typescript files relative to process current working directory #79

Closed lluiscab closed 4 months ago

lluiscab commented 9 months ago

What is the purpose of this pull request? (put an "X" next to item)

What changes did you make? (Give an overview)

I've been having problems with this plugin due to the folder structure of my project and NEXT_TRANSLATE_PATH not being respected by ts.createProgram(). This change makes sure that the typescript Program created can access the source files correctly and does not return undefined.

It might also be a good idea to throw an error/warning if program.getSourceFile() returns undefined, altering the user of the issue. I've been trying to track down the cause of an issue similar to https://github.com/aralroca/next-translate/issues/1040 (different root cause) and it's been a nightmare trying to debug it without having any information available on whatever or not the plugin was working correctly.

I'd recommend merging https://github.com/aralroca/next-translate-plugin/pull/78 first

aralroca commented 4 months ago

@lluiscab currently available in 3.0.0-canary.3 pre-release