plugin-seed icon indicating copy to clipboard operation
plugin-seed copied to clipboard

fix: angular tsconfig wrong @demo/shared path

Open agonper opened this issue 3 years ago • 4 comments

Error description

Angular demo build cannot find files under @demo/shared scope

Steps to reproduce

  1. Download plugin seed
  2. Setup and config
  3. Add a plugin
  4. Try to build the Angular demo app
  5. Webpack will fail with the following error: image

agonper avatar Mar 22 '22 17:03 agonper

Thanks @agonper there's one other adjustment the tools need with latest updates to ensure the angular app's tsconfig is correct, it should instead annotate along with each added package like this for example:

{
  "extends": "../../tsconfig.base.json",
  "compilerOptions": {
    "paths": {
      "~/*": ["src/*"],
      "@demo/shared": ["tools/demo/index.ts"],
      "@nativescript/*": ["packages/*"]
    }
  },
  "files": ["./references.d.ts", "./src/main.ts", "./src/polyfills.ts"],
  "include": ["../../packages/**/references.d.ts", "../../**/*.android.ts", "../../**/*.ios.ts"]
}

That would allow natural livesync changes with Angular demo. I'll add this adjustment in the tools this week.

NathanWalker avatar Mar 23 '22 02:03 NathanWalker

You're welcome @NathanWalker. I'm happy to help 🙂 .

That would allow natural livesync changes with Angular demo.

I noticed that too. Now with Ivy it should no longer be necessary to pre-build Angular plugins for live sync, right? That would be an enormous DX boost!

Looking forward seeing that adjustment! Thanks for making NativeScript better every day 😄

agonper avatar Mar 23 '22 09:03 agonper

Correct! Yes that should allow the angular demo to work/livesync like the vanilla demo. Thanks for the kind words 🤗

NathanWalker avatar Mar 23 '22 15:03 NathanWalker

Any progress on this issue? I have tried implementing the tsconfig changes but there are still errors resolving modules.

jnorkus avatar Apr 20 '22 08:04 jnorkus