Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: angular tsconfig wrong @demo/shared path #25

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

Open
agonper wants to merge 1 commit into NativeScript:main
base: main
Choose a base branch
Loading
from agonper:ng-tsconfig

Conversation

@agonper
Copy link
Contributor

@agonper agonper commented Mar 22, 2022

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

Copy link
Contributor

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.

jkod and fpaaske reacted with thumbs up emoji

Copy link
Contributor Author

agonper commented Mar 23, 2022

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 😄

Copy link
Contributor

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

agonper reacted with hooray emoji

Copy link

jnorkus commented Apr 20, 2022

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

cjohn001 reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /